add medication and triggers to headache logs

Author: crmne

app/controllers/headache_logs_controller.rb

@@ -73,7 +73,7 @@ def set_headache_log
 
     # Only allow a list of trusted parameters through.
     def headache_log_params
-      params.require(:headache_log).permit(:start_time, :end_time, :intensity, :notes)
+      params.require(:headache_log).permit(:start_time, :end_time, :intensity, :notes, :medication, :triggers)
     end
 
     def set_share_link

app/views/headache_logs/_form.html.erb

@@ -26,6 +26,16 @@
     <%= form.number_field :intensity, in: 1..10, class: "mt-1 focus:ring-blue-500 focus:border-blue-500 block w-full shadow-sm sm:text-sm border-gray-300 rounded-md" %>
   </div>
 
+  <div>
+    <%= form.label :medication, class: "block text-sm font-medium text-gray-700" %>
+    <%= form.text_field :medication, class: "mt-1 focus:ring-blue-500 focus:border-blue-500 block w-full shadow-sm sm:text-sm border-gray-300 rounded-md" %>
+  </div>
+
+  <div>
+    <%= form.label :triggers, class: "block text-sm font-medium text-gray-700" %>
+    <%= form.text_area :triggers, class: "mt-1 focus:ring-blue-500 focus:border-blue-500 block w-full shadow-sm sm:text-sm border-gray-300 rounded-md" %>
+  </div>
+
   <div>
     <%= form.label :notes, class: "block text-sm font-medium text-gray-700" %>
     <%= form.text_area :notes, class: "mt-1 focus:ring-blue-500 focus:border-blue-500 block w-full shadow-sm sm:text-sm border-gray-300 rounded-md" %>

app/views/headache_logs/_headache_log.html.erb

@@ -3,10 +3,18 @@
     <p class="font-semibold">Start Time: <%= headache_log.start_time.strftime("%B %d, %Y at %I:%M %p") %></p>
     <p>End Time: <%= headache_log.end_time&.strftime("%B %d, %Y at %I:%M %p") || "Ongoing" %></p>
     <p>Intensity: <span class="font-bold"><%= headache_log.intensity %></span></p>
-    <p>Notes: <%= headache_log.notes %></p>
+    <% if headache_log.medication.present? %>
+      <p>Medication: <%= headache_log.medication %></p>
+    <% end %>
+    <% if headache_log.triggers.present? %>
+      <p>Triggers: <%= headache_log.triggers %></p>
+    <% end %>
+    <% if headache_log.notes.present? %>
+      <p>Notes: <%= headache_log.notes %></p>
+    <% end %>
     <div class="mt-2">
       <%= link_to "Edit", edit_headache_log_path(headache_log), class: "text-blue-500 hover:text-blue-700 mr-2" %>
-      <%= link_to "Delete", headache_log_path(headache_log), data: { turbo_method: :delete, turbo_confirm: "Are you sure?" }, class: "text-red-500 hover:text-red-700" %>
+      <%= button_to "Delete", headache_log_path(headache_log), method: :delete, data: { turbo_confirm: "Are you sure?" }, class: "text-red-500 hover:text-red-700" %>
     </div>
   </div>
 <% end %>

app/views/headache_logs/show.html.erb

@@ -6,6 +6,12 @@
       <p><strong class="font-semibold">Start time:</strong> <%= @headache_log.start_time.strftime("%B %d, %Y at %I:%M %p") %></p>
       <p><strong class="font-semibold">End time:</strong> <%= @headache_log.end_time&.strftime("%B %d, %Y at %I:%M %p") || "Ongoing" %></p>
       <p><strong class="font-semibold">Intensity:</strong> <%= @headache_log.intensity %></p>
+      <% if @headache_log.medication.present? %>
+        <p><strong class="font-semibold">Medication:</strong> <%= @headache_log.medication %></p>
+      <% end %>
+      <% if @headache_log.triggers.present? %>
+        <p><strong class="font-semibold">Triggers:</strong> <%= @headache_log.triggers %></p>
+      <% end %>
       <p><strong class="font-semibold">Notes:</strong> <%= @headache_log.notes %></p>
     </div>
 

app/views/shared_logs/show.html.erb

@@ -1,13 +1,21 @@
 <div class="container mx-auto px-4 py-8">
   <h1 class="text-2xl font-bold mb-4">Headache Logs for <%= @user.email %></h1>
-  
+
   <div class="space-y-4">
-    <% @headache_logs.each do |log| %>
-      <div class="bg-white shadow rounded-lg p-4">
-        <p class="font-semibold">Start Time: <%= log.start_time.strftime("%B %d, %Y at %I:%M %p") %></p>
-        <p>End Time: <%= log.end_time&.strftime("%B %d, %Y at %I:%M %p") || "Ongoing" %></p>
-        <p>Intensity: <span class="font-bold"><%= log.intensity %></span></p>
-        <p>Notes: <%= log.notes %></p>
+    <% @headache_logs.each do |headache_log| %>
+      <div class="bg-white p-4 rounded-lg shadow mb-4">
+        <p class="font-semibold">Start Time: <%= headache_log.start_time.strftime("%B %d, %Y at %I:%M %p") %></p>
+        <p>End Time: <%= headache_log.end_time&.strftime("%B %d, %Y at %I:%M %p") || "Ongoing" %></p>
+        <p>Intensity: <span class="font-bold"><%= headache_log.intensity %></span></p>
+        <% if headache_log.medication.present? %>
+          <p>Medication: <%= headache_log.medication %></p>
+        <% end %>
+        <% if headache_log.triggers.present? %>
+          <p>Triggers: <%= headache_log.triggers %></p>
+        <% end %>
+        <% if headache_log.notes.present? %>
+          <p>Notes: <%= headache_log.notes %></p>
+        <% end %>
       </div>
     <% end %>
   </div>

db/migrate/20240813094347_add_medication_and_triggers_to_headache_logs.rb

@@ -0,0 +1,6 @@
+class AddMedicationAndTriggersToHeadacheLogs < ActiveRecord::Migration[7.2]
+  def change
+    add_column :headache_logs, :medication, :string
+    add_column :headache_logs, :triggers, :text
+  end
+end