avo 2.17.1.pre.2.customauthorizationclients → 2.17.1.pre.4.issue.1342

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e76409a6dd8d5c3a89a467e4cbed5e46f0c292988fd60efa5a1c673be10e36a
-  data.tar.gz: '09bde2e9be219d3f9af5fb3256ee9ed60c77d86d587e8f9131c8d3e95d03ef29'
+  metadata.gz: 0cb67ed800f6808830e072dfeaf5acb53a9c8960d7ca1b1fc01a5ac27ab478b4
+  data.tar.gz: 861965614539773f8c6242e211bb7e3449b8393ad2306cf3aa1bcbb030a36212
 SHA512:
-  metadata.gz: 82ceba1946394ea1bea5cf4ef8283d75ddb40fdcfcfdd55ee19cad8aff34ed873698442dd846711134e6d961e288ca723cf8f9fab0aab122c0a49319e432ff05
-  data.tar.gz: 3829a9ec1348318c537aee5954463c41864ee710790a8a333c2f075102d205ec90e50e70f766051aa2e892c2d0d8609b0bedc299a502d557a0806e9c34107269
+  metadata.gz: 5a2d565a8a5839e25f23cd4bdac2308c07a5b89dacadd090b8ddcaf5180b58fa4d7f844ebafbd02c167d60d9f0f5193c6b7cd2790c61f538fc8aadfa50e6f32b
+  data.tar.gz: e1a6e356003227ff73440bd51a8aecdce3f4522be2c4cb0829fbd35a88d9916ffcb860638428f19296d66e054396f8683f759852282c22559b6f76fd415a8d81

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    avo (2.17.1.pre.2.customauthorizationclients)
+    avo (2.17.1.pre.4.issue.1342)
       active_link_to
       addressable
       breadcrumbs_on_rails
@@ -17,7 +17,7 @@ PATH
       pundit
       rails (>= 6.0)
       turbo-rails
-      view_component (= 2.60)
+      view_component
       zeitwerk
 
 GEM
@@ -265,7 +265,7 @@ GEM
       net-protocol
       timeout
     nio4r (2.5.8)
-    nokogiri (1.13.8)
+    nokogiri (1.13.9)
       mini_portile2 (~> 2.8.0)
       racc (~> 1.4)
     orm_adapter (0.5.0)
@@ -413,8 +413,9 @@ GEM
     tzinfo (2.0.5)
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.2.0)
-    view_component (2.60.0)
+    view_component (2.74.1)
       activesupport (>= 5.0.0, < 8.0)
+      concurrent-ruby (~> 1.0)
       method_source (~> 1.0)
     warden (1.2.9)
       rack (>= 2.0.9)
@@ -437,7 +438,7 @@ GEM
     websocket-extensions (0.1.5)
     xpath (3.2.0)
       nokogiri (~> 1.8)
-    zeitwerk (2.6.0)
+    zeitwerk (2.6.1)
 
 PLATFORMS
   ruby

data/app/components/avo/button_component.rb

@@ -88,8 +88,8 @@ class Avo::ButtonComponent < ViewComponent::Base
   end
 
   def output_button
-    if @args[:method].present?
-      button_to @args[:url], **args do
+    if args.dig(:method).present? || args.dig(:data, :turbo_method).present?
+      button_to args[:url], **args do
         full_content
       end
     else

data/app/components/avo/fields/common/single_file_viewer_component.html.erb

@@ -36,15 +36,15 @@
         <% if @resource.authorization.authorize_action(:delete_attachments?, raise_exception: false) %>
           <%= a_link destroy_path,
             icon: 'heroicons/outline/trash',
-            method: :delete,
             color: :red,
             compact: true,
             size: :xs,
             class: 'text-center',
             title: t('avo.delete_file', item: file.filename),
             data: {
-              'turbo-frame': 'destroy_attachment_form',
-              confirm: t('avo.are_you_sure'),
+              turbo_method: :delete,
+              turbo_frame: 'destroy_attachment_form',
+              turbo_confirm: t('avo.are_you_sure'),
               tippy: :tooltip
             } %>
         <% end %>

data/app/components/avo/fields/date_field/edit_component.html.erb

@@ -7,6 +7,8 @@
       date_field_picker_format_value: @field.picker_format,
       date_field_first_day_of_week_value: @field.first_day_of_week,
       date_field_disable_mobile_value: @field.disable_mobile,
+      date_field_field_type_value: "date",
+      date_field_picker_options_value: @field.picker_options,
   } do %>
     <%= datetime_field "fake_#{@field.id}", "fake",
       value: @field.edit_formatted_value,

data/app/components/avo/fields/date_field/index_component.html.erb

@@ -3,6 +3,7 @@
     controller: "date-field",
     date_field_view_value: @view,
     date_field_format_value: @field.format,
+    date_field_field_type_value: "date",
   } do %>
     <%= @field.formatted_value %>
   <% end %>

data/app/components/avo/fields/date_field/show_component.html.erb

@@ -3,6 +3,7 @@
     controller: "date-field",
     date_field_view_value: @view,
     date_field_format_value: @field.format,
+    date_field_field_type_value: "date",
   } do %>
     <%= @field.formatted_value %>
   <% end %>

data/app/components/avo/fields/date_time_field/edit_component.html.erb

@@ -8,6 +8,9 @@
       date_field_disable_mobile_value: @field.disable_mobile,
       date_field_time24_hr_value: @field.time_24hr,
       date_field_timezone_value: @field.timezone,
+      date_field_relative_value: @field.relative,
+      date_field_field_type_value: "dateTime",
+      date_field_picker_options_value: @field.picker_options,
   } do %>
     <%= datetime_field "fake_#{@field.id}", "fake",
       value: @field.edit_formatted_value,

data/app/components/avo/fields/date_time_field/index_component.html.erb

@@ -5,7 +5,9 @@
     date_field_enable_time_value: true,
     date_field_format_value: @field.format,
     date_field_timezone_value: @field.timezone,
+    date_field_relative_value: @field.relative,
     date_field_picker_format_value: @field.picker_format,
+    date_field_field_type_value: "dateTime",
   } do %>
     <%= @field.formatted_value %>
   <% end %>

data/app/components/avo/fields/date_time_field/show_component.html.erb

@@ -5,7 +5,9 @@
     date_field_enable_time_value: true,
     date_field_format_value: @field.format,
     date_field_timezone_value: @field.timezone,
+    date_field_relative_value: @field.relative,
     date_field_picker_format_value: @field.picker_format,
+    date_field_field_type_value: "dateTime",
   } do %>
     <%= @field.formatted_value %>
   <% end %>

data/app/components/avo/fields/time_field/edit_component.html.erb

@@ -0,0 +1,40 @@
+<%= field_wrapper **field_wrapper_args do %>
+  <%= content_tag :div, data: {
+      controller: "date-field",
+      date_field_view_value: @view,
+      date_field_enable_time_value: true,
+      date_field_picker_format_value: @field.picker_format,
+      date_field_disable_mobile_value: @field.disable_mobile,
+      date_field_time24_hr_value: @field.time_24hr,
+      date_field_no_calendar_value: true,
+      date_field_timezone_value: @field.timezone,
+      date_field_relative_value: @field.relative,
+      date_field_field_type_value: "time",
+      date_field_picker_options_value: @field.picker_options,
+  } do %>
+    <%= datetime_field "fake_#{@field.id}", "fake",
+      value: @field.edit_formatted_value,
+      class: classes("w-full"),
+      data: {
+        'date-field-target': 'fakeInput',
+        placeholder: @field.placeholder,
+        **@field.get_html(:data, view: view, element: :input)
+      },
+      disabled: @field.is_readonly?,
+      placeholder: @field.placeholder,
+      style: @field.get_html(:style, view: view, element: :input)
+    %>
+    <%= @form.text_field @field.id,
+      value: @field.edit_formatted_value,
+      class: classes("w-full hidden"),
+      data: {
+        'date-field-target': 'input',
+        placeholder: @field.placeholder,
+        **@field.get_html(:data, view: view, element: :input)
+      },
+      disabled: @field.is_readonly?,
+      placeholder: @field.placeholder,
+      style: @field.get_html(:style, view: view, element: :input)
+    %>
+  <% end %>
+<% end %>

data/app/components/avo/fields/time_field/edit_component.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+class Avo::Fields::TimeField::EditComponent < Avo::Fields::EditComponent
+end

data/app/components/avo/fields/time_field/index_component.html.erb

@@ -0,0 +1,15 @@
+<%= index_field_wrapper **field_wrapper_args do %>
+   <%= content_tag :div, data: {
+    controller: "date-field",
+    date_field_view_value: @view,
+    date_field_enable_time_value: true,
+    date_field_format_value: @field.format,
+    date_field_timezone_value: @field.timezone,
+    date_field_picker_format_value: @field.picker_format,
+    date_field_no_calendar_value: true,
+    date_field_relative_value: @field.relative,
+    date_field_field_type_value: "time",
+  } do %>
+    <%= @field.formatted_value %>
+  <% end %>
+<% end %>

data/app/components/avo/fields/time_field/index_component.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+class Avo::Fields::TimeField::IndexComponent < Avo::Fields::IndexComponent
+end

data/app/components/avo/fields/time_field/show_component.html.erb

@@ -0,0 +1,15 @@
+<%= field_wrapper **field_wrapper_args do %>
+  <%= content_tag :div, data: {
+    controller: "date-field",
+    date_field_view_value: @view,
+    date_field_enable_time_value: true,
+    date_field_format_value: @field.format,
+    date_field_timezone_value: @field.timezone,
+    date_field_picker_format_value: @field.picker_format,
+    date_field_no_calendar_value: true,
+    date_field_relative_value: @field.relative,
+    date_field_field_type_value: "time",
+  } do %>
+    <%= @field.formatted_value %>
+  <% end %>
+<% end %>

data/app/components/avo/fields/time_field/show_component.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+class Avo::Fields::TimeField::ShowComponent < Avo::Fields::ShowComponent
+end

data/app/components/avo/index/resource_controls_component.html.erb

@@ -42,7 +42,7 @@
         type: :submit,
         data: {
           target: 'control:detach',
-          confirm: t('avo.are_you_sure_detach_item', item: singular_resource_name),
+          turbo_confirm: t('avo.are_you_sure_detach_item', item: singular_resource_name),
           control: :detach,
           'resource-id': @resource.model.id,
           'tippy': 'tooltip',
@@ -66,7 +66,7 @@
         type: :submit,
         data: {
           target: 'control:destroy',
-          confirm: t('avo.are_you_sure',  item: singular_resource_name),
+          turbo_confirm: t('avo.are_you_sure',  item: singular_resource_name),
           control: :destroy,
           'resource-id': @resource.model.id,
           'tippy': 'tooltip',

data/app/components/avo/panel_component.html.erb

@@ -25,10 +25,8 @@
   <% end %>
   <% if body? %>
     <div class="flex flex-col sm:flex-row space-y-4 sm:space-y-0 sm:gap-4 w-full">
-      <div class="flex-1 overflow-auto <% if sidebar? %> w-2/3 <% end %>">
-        <div class="relative <%= white_panel_classes %> <%= @body_classes %>">
-          <%= body %>
-        </div>
+      <div class="relative flex-1 <%= white_panel_classes %> <%= @body_classes %> <% if sidebar? %> w-2/3 overflow-auto <% else %> w-full <% end %>">
+        <%= body %>
       </div>
       <% if sidebar? %>
         <div class="w-full sm:w-1/3 flex-shrink-0 h-full <%= white_panel_classes %>">

data/app/components/avo/panel_component.rb

@@ -5,6 +5,8 @@ class Avo::PanelComponent < ViewComponent::Base
   attr_reader :name
   attr_reader :classes
 
+  delegate :white_panel_classes, to: :helpers
+
   renders_one :tools
   renders_one :body
   renders_one :sidebar
@@ -26,10 +28,6 @@ class Avo::PanelComponent < ViewComponent::Base
 
   private
 
-  def white_panel_classes
-    "bg-white rounded shadow"
-  end
-
   def data_attributes
     @data.merge({"panel-index": @index})
   end

data/app/components/avo/sidebar_profile_component.html.erb

@@ -37,8 +37,11 @@
         <%# Example link below %>
         <%#= render Avo::ProfileItemComponent.new label: 'Profile', path: '/profile', icon: 'user-circle' %>
         <%= button_to helpers.main_app.send(destroy_user_session_path),
-          method: :delete,
           form: { "data-turbo" => "false" },
+          method: :delete,
+          data: {
+            confirm: t('avo.are_you_sure')
+          },
           class: "flex-1 flex items-center justify-center bg-white text-left cursor-pointer text-red-600 font-semibold hover:bg-red-100 block px-4 py-1 w-full py-3 text-center rounded w-full",
           form_class: 'flex-1' do %>
           <%= helpers.svg 'logout', class: 'h-4 mr-1' %> <%= t('avo.sign_out') %>

data/app/components/avo/tab_switcher_component.html.erb

@@ -20,7 +20,7 @@
     </div>
   </div>
 <% else %>
-  <div class="flex flex-wrap gap-2 bg-white p-2" data-target="tab-switcher" data-style="pills">
+  <div class="flex flex-wrap gap-2 p-2 <%= white_panel_classes %>" data-target="tab-switcher" data-style="pills">
     <% visible_items.each do |tab| %>
       <%= a_link tab_path(tab),
       color: selected?(tab) ? :primary : :gray,

data/app/components/avo/tab_switcher_component.rb

@@ -11,6 +11,8 @@ class Avo::TabSwitcherComponent < Avo::BaseComponent
   attr_reader :view
   attr_reader :style
 
+  delegate :white_panel_classes, to: :helpers
+
   def initialize(resource:, group:, current_tab:, active_tab_name:, view:, style:)
     @active_tab_name = active_tab_name
     @resource = resource

data/app/components/avo/views/resource_edit_component.html.erb

@@ -25,18 +25,16 @@
           <% end %>
           <% if can_see_the_destroy_button? %>
             <%= a_link destroy_path,
-            method: :delete,
-            local: true,
-            style: :text,
-            loading: true,
-            confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
-            color: :red,
-            icon: 'trash',
-            form_class: 'flex flex-col sm:flex-row sm:inline-flex',
-            data: {
-              control: :destroy,
-              'resource-id': @resource.model.id,
-            } do %>
+              style: :text,
+              color: :red,
+              icon: 'trash',
+              form_class: 'flex flex-col sm:flex-row sm:inline-flex',
+              data: {
+                turbo_confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
+                turbo_method: :delete,
+                control: :destroy,
+                'resource-id': @resource.model.id,
+              } do %>
               <%= t('avo.delete').capitalize %>
             <% end %>
           <% end %>

data/app/components/avo/views/resource_index_component.html.erb

@@ -46,10 +46,8 @@
       </div>
       <% if view_type.to_sym == :table %>
         <% if @resources.present? %>
-          <div class="w-full overflow-auto flex flex-col mt-0 mac-styled-scrollbar">
-            <div class="relative flex-1 flex">
-              <%= render(Avo::Index::ResourceTableComponent.new(resources: @resources, resource: @resource, reflection: @reflection, parent_model: @parent_model, parent_resource: @parent_resource, pagy: @pagy, query: @query)) %>
-            </div>
+          <div class="w-full relative flex-1 flex mac-styled-scrollbar overflow-auto mt-0">
+            <%= render(Avo::Index::ResourceTableComponent.new(resources: @resources, resource: @resource, reflection: @reflection, parent_model: @parent_model, parent_resource: @parent_resource, pagy: @pagy, query: @query)) %>
           </div>
         <% else %>
           <%= helpers.empty_state resource_name: @resource.name.downcase.pluralize, related_name: params[:related_name], view_type: view_type, add_background: true %>

data/app/components/avo/views/resource_show_component.html.erb

@@ -21,17 +21,15 @@
             <% end %>
           <% elsif control.delete_button? %>
             <% if can_see_the_destroy_button? %>
-              <%= a_button url: helpers.resource_path(model: @resource.model, resource: @resource),
-                method: :delete,
-                local: true,
+              <%= a_link helpers.resource_path(model: @resource.model, resource: @resource),
                 style: :text,
-                loading: true,
-                confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
                 color: :red,
                 icon: 'trash',
                 form_class: 'flex flex-col sm:flex-row sm:inline-flex',
                 title: control.title,
                 data: {
+                  turbo_confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
+                  turbo_method: :delete,
                   control: :destroy,
                   tippy: control.title ? :tooltip : nil,
                   'resource-id': @resource.model.id,
@@ -84,13 +82,13 @@
           <% elsif control.detach_button? %>
             <% if @reflection.present? && @resource.model.present? && can_detach? %>
               <%= a_button url: detach_path,
-                  icon: 'detach',
-                  method: :delete,
-                  form_class: 'flex flex-col sm:flex-row sm:inline-flex',
-                  style: :text,
-                  data: {
-                    confirm: "Are you sure you want to detach this #{title}."
-                  } do %>
+                icon: 'detach',
+                method: :delete,
+                form_class: 'flex flex-col sm:flex-row sm:inline-flex',
+                style: :text,
+                data: {
+                  confirm: "Are you sure you want to detach this #{title}."
+                } do %>
                 <%= control.label %>
               <% end %>
             <% end %>
@@ -126,19 +124,17 @@
             <%= t('avo.go_back') %>
           <% end %>
           <% if can_see_the_destroy_button? %>
-            <%= a_button url: destroy_path,
-                method: :delete,
-                local: true,
-                style: :text,
-                loading: true,
-                confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
-                color: :red,
-                icon: 'trash',
-                form_class: 'flex flex-col sm:flex-row sm:inline-flex',
-                data: {
-                  control: :destroy,
-                  'resource-id': @resource.model.id,
-                } do %>
+            <%= a_link destroy_path,
+              style: :text,
+              color: :red,
+              icon: 'trash',
+              form_class: 'flex flex-col sm:flex-row sm:inline-flex',
+              data: {
+                turbo_confirm: t('avo.are_you_sure', item: @resource.model.model_name.name.downcase),
+                turbo_method: :delete,
+                control: :destroy,
+                'resource-id': @resource.model.id,
+              } do %>
               <%= t('avo.delete').capitalize %>
             <% end %>
           <% end %>

data/app/helpers/avo/application_helper.rb

@@ -89,6 +89,10 @@ module Avo
       classes
     end
 
+    def white_panel_classes
+      "bg-white rounded shadow-md"
+    end
+
     def get_model_class(model)
       if model.instance_of?(Class)
         model

data/app/javascript/js/controllers/fields/date_field_controller.js

@@ -7,6 +7,9 @@ function universalTimestamp(timestampStr) {
   return new Date(new Date(timestampStr).getTime() + (new Date(timestampStr).getTimezoneOffset() * 60 * 1000))
 }
 
+const RAW_DATE_FORMAT = 'y/LL/dd'
+const RAW_TIME_FORMAT = 'TT'
+
 export default class extends Controller {
   static targets = ['input', 'fakeInput']
 
@@ -19,6 +22,10 @@ export default class extends Controller {
     firstDayOfWeek: Number,
     time24Hr: Boolean,
     disableMobile: Boolean,
+    noCalendar: Boolean,
+    relative: Boolean,
+    fieldType: { type: String, default: 'dateTime' },
+    pickerOptions: { type: Object, default: {} },
   }
 
   flatpickrInstance;
@@ -53,6 +60,18 @@ export default class extends Controller {
     return this.viewValue === 'show'
   }
 
+  get fieldIsDate() {
+    return this.fieldTypeValue === 'date'
+  }
+
+  get fieldIsDateTime() {
+    return this.fieldTypeValue === 'dateTime'
+  }
+
+  get fieldIsTime() {
+    return this.fieldTypeValue === 'time'
+  }
+
   // Parse the time as if it were UTC
   get parsedValue() {
     return DateTime.fromISO(this.initialValue, { zone: 'UTC' })
@@ -90,8 +109,8 @@ export default class extends Controller {
   initShow() {
     let value = this.parsedValue
 
-    // Set the zone only if the type of field is date time.
-    if (this.enableTimeValue) {
+    // Set the zone only if the type of field is date time or relative time.
+    if (this.enableTimeValue && this.relativeValue) {
       value = value.setZone(this.displayTimezone)
     }
 
@@ -109,6 +128,8 @@ export default class extends Controller {
       },
       altInput: true,
       onChange: this.onChange.bind(this),
+      noCalendar: false,
+      ...this.pickerOptionsValue,
     }
 
     // Set the format of the displayed input field.
@@ -124,24 +145,45 @@ export default class extends Controller {
     options.enableTime = this.enableTimeValue
     options.enableSeconds = this.enableTimeValue
 
-    // enable timezone display
-    if (this.enableTimeValue) {
-      options.defaultDate = this.parsedValue.setZone(this.displayTimezone).toISO()
+    // Hide calendar and only keep time picker.
+    options.noCalendar = this.noCalendarValue
 
-      options.dateFormat = 'Y-m-d H:i:S'
-    } else {
-      // Because the browser treats the date like a timestamp and updates it at 00:00 hour, when on a western timezone the date will be converted with one day offset.
-      // Ex: 2022-01-30 will render as 2022-01-29 on an American timezone
-      options.defaultDate = universalTimestamp(this.initialValue)
+    if (this.initialValue) {
+      // Enable timezone display
+      if (this.enableTimeValue && this.relativeValue) {
+        options.defaultDate = this.parsedValue.setZone(this.displayTimezone).toISO()
+
+        options.dateFormat = 'Y-m-d H:i:S'
+      } else {
+        // Because the browser treats the date like a timestamp and updates it at 00:00 hour, when on a western timezone the date will be converted with one day offset.
+        // Ex: 2022-01-30 will render as 2022-01-29 on an American timezone
+        options.defaultDate = universalTimestamp(this.initialValue)
+      }
     }
 
     this.flatpickrInstance = flatpickr(this.fakeInputTarget, options)
 
-    if (this.enableTimeValue) {
-      this.updateRealInput(this.parsedValue.setZone(this.displayTimezone).toISO())
-    } else {
-      this.updateRealInput(universalTimestamp(this.initialValue))
+    // Don't try to parse the value if the input is empty.
+    if (!this.initialValue) {
+      return
     }
+
+    let value
+    switch (this.fieldTypeValue) {
+      case 'time':
+        // For time values, we should maintain the real value and format it to a time-friendly format.
+        value = this.parsedValue.setZone(this.displayTimezone, { keepLocalTime: true }).toFormat(RAW_TIME_FORMAT)
+        break
+      case 'date':
+        value = DateTime.fromJSDate(universalTimestamp(this.initialValue)).toFormat(RAW_DATE_FORMAT)
+        break
+      default:
+      case 'dateTime':
+        value = this.parsedValue.setZone(this.displayTimezone).toISO()
+        break
+    }
+
+    this.updateRealInput(value)
   }
 
   onChange(selectedDates) {
@@ -152,24 +194,34 @@ export default class extends Controller {
       return
     }
 
-    let time
     let args = {}
 
-    if (this.timezoneValue) {
+    // For values that involve time we should keep the local time.
+    if (this.timezoneValue || !this.relativeValue) {
       args = { keepLocalTime: true }
     } else {
       args = { keepLocalTime: false }
     }
 
-    if (this.enableTimeValue) {
-      time = DateTime.fromISO(selectedDates[0].toISOString()).setZone('UTC', args)
-    } else {
-      time = DateTime.fromISO(selectedDates[0].toISOString()).setZone('UTC', { keepLocalTime: true })
+    let value
+    switch (this.fieldTypeValue) {
+      case 'time':
+        // For time values, we should maintain the real value and format it to a time-friendly format.
+        value = DateTime.fromISO(selectedDates[0].toISOString()).setZone('UTC', args).toFormat(RAW_TIME_FORMAT)
+        break
+      case 'date':
+        value = DateTime.fromISO(selectedDates[0].toISOString()).setZone('UTC', { keepLocalTime: true }).toFormat(RAW_DATE_FORMAT)
+        break
+      default:
+      case 'dateTime':
+        value = DateTime.fromISO(selectedDates[0].toISOString()).setZone('UTC', args).toISO()
+        break
     }
 
-    this.updateRealInput(time)
+    this.updateRealInput(value)
   }
 
+  // Value should be a string
   updateRealInput(value) {
     this.inputTarget.value = value
   }

data/app/javascript/js/controllers/search_controller.js

@@ -73,6 +73,9 @@ export default class extends Controller {
       Mousetrap.bind(['command+k', 'ctrl+k'], () => this.showSearchPanel())
     }
 
+    // This line fixes a bug where the search box would be duplicated on back navigation.
+    this.autocompleteTarget.innerHTML = ''
+
     autocomplete({
       container: this.autocompleteTarget,
       placeholder: this.translationKeys.placeholder,

data/avo.gemspec

@@ -40,7 +40,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "httparty"
   spec.add_dependency "active_link_to"
   spec.add_dependency "image_processing"
-  spec.add_dependency "view_component", "2.60"
+  spec.add_dependency "view_component"
   spec.add_dependency "turbo-rails"
   spec.add_dependency "addressable"
   spec.add_dependency "meta-tags"

data/db/factories.rb

@@ -69,6 +69,7 @@ FactoryBot.define do
     skills { [Faker::Educator.subject, Faker::Educator.subject, Faker::Educator.subject, Faker::Educator.subject, Faker::Educator.subject] }
     country { Course.countries.sample }
     city { Course.cities.stringify_keys[country].sample }
+    starting_at { Time.now }
   end
 
   factory :course_link, class: "Course::Link" do