Add flash

renuo · Nov 5, 2024 · ed36769 · ed36769
1 parent 738d308
commit ed36769
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 4 deletions.
diff --git a/app/controllers/hotsheet/pages_controller.rb b/app/controllers/hotsheet/pages_controller.rb
@@ -13,11 +13,17 @@ def broadcast_edit_intent
                                    })
     end
 
-    def update
-      record = model.find(params[:id])
-      notice = "#{record} #{record.update(model_params) ? "updated successfully" : "update failed"}"
+    def update # rubocop:disable Metrics/AbcSize
+      record = model.find params[:id]
 
-      redirect_to polymorphic_path(record), notice: notice
+      if record.update model_params
+        flash[:notice] = t("hotsheet.success", record: record)
+      else
+        flash[:alert] = t("hotsheet.error", record: record,
+                                            errors: record.errors.full_messages.join(", "))
+      end
+
+      redirect_to "#{root_path}#{model.table_name}"
     end
 
     private

diff --git a/app/views/layouts/hotsheet/_flash.html.erb b/app/views/layouts/hotsheet/_flash.html.erb
@@ -0,0 +1,7 @@
+<%# locals: () %>
+
+<% flash.each do |type, msg| %>
+  <div class='flash <%= type %>'>
+    <span><%= msg %></span>
+  </div>
+<% end %>
diff --git a/app/views/layouts/hotsheet/_sidebar.html.erb b/app/views/layouts/hotsheet/_sidebar.html.erb
@@ -1,3 +1,5 @@
+<%# locals: () %>
+
 <aside class="sidebar">
   <nav>
     <ul class="nav-list">

diff --git a/app/views/layouts/hotsheet/application.html.erb b/app/views/layouts/hotsheet/application.html.erb
@@ -11,6 +11,7 @@
   </head>
   <body>
     <%= render "layouts/hotsheet/sidebar" %>
+    <%= render "layouts/hotsheet/flash" %>
     <main>
       <%= yield %>
     </main>

diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -1,2 +1,5 @@
 en:
   home: Home
+  hotsheet:
+    success: '%{record} updated successfully'
+    error: '%{record} update failed: %{errors}'
-Original file line number
+Diff line change
@@ Expand Up / @@ -11,6 +11,7 @@ @@
       </head>
       <body>
         <%= render "layouts/hotsheet/sidebar" %>
+        <%= render "layouts/hotsheet/flash" %>
         <main>
           <%= yield %>
         </main>
@@ Expand Down @@