Rename flex to strata: controllers (#194)

baonguyenNava · web-flow · commit f6f963bf671c · 2025-09-25T07:09:33.000-06:00
## Changes

Rename flex module to strata for controllers only. Views were also moved to their new folders.
diff --git a/app/controllers/strata/application_controller.rb b/app/controllers/strata/application_controller.rb
@@ -1,4 +1,4 @@
-module Flex
+module Strata
   class ApplicationController < ::ApplicationController
   end
 end
diff --git a/app/controllers/strata/staff_controller.rb b/app/controllers/strata/staff_controller.rb
@@ -1,8 +1,8 @@
-module Flex
+module Strata
   # Base controller for all staff-related functionality.
-  # Sets the layout to the flex/staff layout
+  # Sets the layout to the strata/staff layout
   class StaffController < ApplicationController
-    layout "flex/staff"
+    layout "strata/staff"
 
     before_action :set_header_cases_links
 
diff --git a/app/controllers/strata/tasks_controller.rb b/app/controllers/strata/tasks_controller.rb
@@ -1,4 +1,4 @@
-module Flex
+module Strata
   # Controller for managing Flex::Task records. Handles listing, filtering, showing, and updating tasks.
   # This controller helps a parent application manage tasks by not forcing the parent application to implement the same functionality.
   class TasksController < ::StaffController
diff --git a/app/helpers/strata/date_helper.rb b/app/helpers/strata/date_helper.rb
@@ -1,4 +1,4 @@
-module Flex
+module Strata
   # DateHelper provides view helpers for formatting and manipulating dates.
   # This module is used in views to display dates in consistent formats.
   #
diff --git a/app/previews/flex/staff/header_preview.rb b/app/previews/flex/staff/header_preview.rb
@@ -17,8 +17,8 @@ def default
         # Mock the request object in order to simulate the current path
         request = Request.new("/passport_cases/closed")
 
-        # render template: "flex/previews/empty", locals: {
-        render template: "flex/staff/_header", locals: {
+        # render template: "strata/previews/empty", locals: {
+        render template: "strata/staff/_header", locals: {
           cases_links: [
             { name: "Passport Cases", path: "/staff/passport_cases" }
           ],
diff --git a/app/views/flex/shared/name_form.html.erb b/app/views/flex/shared/name_form.html.erb
diff --git a/app/views/flex/tasks/show.html.erb b/app/views/flex/tasks/show.html.erb
@@ -29,7 +29,7 @@
         }
       ]
     %>
-    <%= render partial: "flex/shared/breadcrumbs", locals: { breadcrumbs: } %>
+    <%= render partial: "strata/shared/breadcrumbs", locals: { breadcrumbs: } %>
 
     <h1 class="section-heading-h1"><%= t("tasks.types.#{task.type.underscore}") %></h1>
     <div class="border-bottom-1px margin-bottom-1 padding-bottom-1">
diff --git a/app/views/layouts/strata/staff.html.erb b/app/views/layouts/strata/staff.html.erb
@@ -10,7 +10,7 @@
   </head>
   <body>
     <div class="display-flex flex-column minh-viewport">
-      <%= render partial: 'flex/staff/header' %>
+      <%= render partial: 'strata/staff/header' %>
 
       <main id="main-content" class="grid-col-fill display-flex flex-column">
         <div class="grid-col-fill <%= yield :main_col_class %>">
diff --git a/app/views/strata/cases/_nav_tab.html.erb b/app/views/strata/cases/_nav_tab.html.erb
@@ -0,0 +1,7 @@
+<%= link_to name, path, class: class_names(
+  "text-no-underline padding-y-1 padding-x-2 font-sans-md",
+  {
+    "text-bold border-bottom-05 border-primary text-primary" => active,
+    "text-ink" => !active
+  }
+) %>
diff --git a/app/views/strata/shared/_breadcrumbs.html.erb b/app/views/strata/shared/_breadcrumbs.html.erb
@@ -0,0 +1,18 @@
+<% if content_for?(:breadcrumbs) %>
+  <%= yield :breadcrumbs %>
+<% else %>
+  <nav class="usa-breadcrumb" aria-label="Breadcrumbs">
+    <ol class="usa-breadcrumb__list">
+      <% breadcrumbs[0...-1].each do |breadcrumb| %>
+        <li class="usa-breadcrumb__list-item">
+          <%= link_to breadcrumb[:text], breadcrumb[:link], class: "usa-breadcrumb__link" %>
+        </li>
+      <% end %>
+      <% if breadcrumbs.last %>
+        <li class="usa-breadcrumb__list-item usa-current" aria-current="page">
+          <span><%= breadcrumbs.last[:text] %></span>
+        </li>
+      <% end %>
+    </ol>
+  </nav>
+<% end %>
diff --git a/app/views/strata/shared/_step_indicator.html.erb b/app/views/strata/shared/_step_indicator.html.erb
@@ -0,0 +1,52 @@
+<%
+# Renders a step indicator that shows progress through a multi-step process.
+#
+# @param steps [Array<String, Symbol>] An array of step identifiers that will be displayed in order
+# @param current_step [String, Symbol] The identifier of the current active step
+# @param translation_scope [String] - I18n scope prefix, defaulting to 'flex.application_forms.steps'
+#
+# Each step's display name is fetched from the i18n translations under flex.application_forms.steps
+# Steps before and including the current step are marked as complete.
+
+steps = steps.map(&:to_sym)
+current_step = current_step.to_sym
+current_step_index = steps.index(current_step) || -1
+translation_scope ||= "flex.application_forms.steps"
+step_indicators = steps.map.with_index do |step, index|
+  {
+    name: t("#{ step }", scope: translation_scope, default: step.to_s.humanize),
+    complete: index <= current_step_index,
+    current: step == current_step
+  }
+end
+step_indicator_class = ""
+step_indicator_class += "usa-step-indicator--counters" if local_assigns[:type] == :counters
+%>
+
+<div
+  class="usa-step-indicator <%= step_indicator_class %>"
+  aria-label="progress"
+>
+  <ol class="usa-step-indicator__segments">
+    <% step_indicators.each do |step_indicator| %>
+      <li
+        class="<%= class_names(
+          "usa-step-indicator__segment",
+          "usa-step-indicator__segment--complete": step_indicator[:complete],
+          "usa-step-indicator__segment--current": step_indicator[:current]
+        )%>"
+      >
+          <span class="usa-step-indicator__segment-label">
+            <%= step_indicator[:name] %>
+            <span class="usa-sr-only">
+              <% if step_indicator[:complete] %>
+                completed
+              <% else %>
+                not completed
+              <% end %>
+            </span>
+          </span>
+        </li>
+    <% end %>
+  </ol>
+</div>
diff --git a/app/views/strata/staff/_header.html.erb b/app/views/strata/staff/_header.html.erb
@@ -33,7 +33,7 @@
     <div class="usa-navbar desktop:width-mobile-lg">
       <div class="usa-logo desktop:margin-top-2">
         <em class="usa-logo__text">
-          <%= link_to t('flex.layouts.staff.header.title'), main_app.staff_path, class: 'usa-logo__link' %>
+          <%= link_to t('strata.layouts.staff.header.title'), main_app.staff_path, class: 'usa-logo__link' %>
         </em>
       </div>
       <button class="usa-menu-btn" type="button">Menu</button>
diff --git a/app/views/strata/tasks/_due_date_filter.html.erb b/app/views/strata/tasks/_due_date_filter.html.erb
@@ -0,0 +1,14 @@
+<%= form_with method: 'get' do |form| %>
+  <%= hidden_field_tag(:filter_type, params[:filter_type]) if params[:filter_type].present? %>
+  <%= hidden_field_tag(:filter_status, params[:filter_status]) if params[:filter_status].present? %>
+  <%= form.label :filter_date, t("flex.tasks.partials.due_date_filter.label"), class: 'usa-label' %>
+  <%= form.select :filter_date, [
+      [t("flex.tasks.partials.due_date_filter.options.all"), ""],
+      [t("flex.tasks.partials.due_date_filter.options.overdue"), "overdue"],
+      [t("flex.tasks.partials.due_date_filter.options.today"), "today"],
+      [t("flex.tasks.partials.due_date_filter.options.tomorrow"), "tomorrow"],
+      [t("flex.tasks.partials.due_date_filter.options.this_week"), "this_week"],
+    ],
+    { include_blank: false, selected: params[:filter_date] },
+    { class: 'usa-select', onchange: 'this.form.submit()' } %>
+<% end %>
diff --git a/app/views/strata/tasks/_no_tasks_alert.html.erb b/app/views/strata/tasks/_no_tasks_alert.html.erb
@@ -0,0 +1,7 @@
+<div class="usa-alert usa-alert--info usa-alert--no-icon">
+  <div class="usa-alert__body">
+    <p class="usa-alert__text">
+      <%= t("flex.tasks.partials.no_tasks_alert.message") %>
+    </p>
+  </div>
+</div>
diff --git a/app/views/strata/tasks/_status_tag.html.erb b/app/views/strata/tasks/_status_tag.html.erb
@@ -0,0 +1 @@
+<span class="usa-tag text-ink bg-yellow"><%= status %></span></p>
diff --git a/app/views/strata/tasks/_type_filter.html.erb b/app/views/strata/tasks/_type_filter.html.erb
@@ -0,0 +1,11 @@
+<%= form_with method: 'get' do |form| %>
+  <%= hidden_field_tag(:filter_date, params[:filter_date]) if params[:filter_date].present? %>
+  <%= hidden_field_tag(:filter_status, params[:filter_status]) if params[:filter_status].present? %>
+  <%= form.label :filter_type, t("flex.tasks.partials.type_filter.label"), class: 'usa-label' %>
+  <%= form.select :filter_type, \
+    [[t("flex.tasks.partials.type_filter.options.all"), ""]].append(
+      *task_types.map { |type| [t("tasks.types.#{type.underscore}"), type] }
+    ),
+    { include_blank: false, selected: params[:filter_type] },
+    { class: 'usa-select', onchange: 'this.form.submit()' } %>
+<% end %>
diff --git a/app/views/strata/tasks/index.html.erb b/app/views/strata/tasks/index.html.erb
@@ -0,0 +1,101 @@
+<%# Index view for displaying a list of tasks with filtering and tab navigation
+  #
+  # @param tasks [Array<Flex::Task>] Array of tasks to display
+  # @param unassigned_tasks [Array<Flex::Task>] Array of unassigned tasks for button state
+  # @param task_types [Array<String>] Array of task types for filtering
+  #
+  # @example Task types array
+  #   ["Flex::Task", "Flex::OtherTask", "MyCustomTask"]
+  #
+  # @note Internationalization keys that can be overridden:
+  #   - flex.tasks.index.title: Title for the tasks index page
+  #   - flex.tasks.index.tabs.assigned: Label for the assigned tasks tab
+  #   - flex.tasks.index.tabs.completed: Label for the completed tasks tab
+  #
+ %>
+<!-- Main content -->
+<main class="usa-section">
+  <div class="grid-container">
+    <h1 class="usa-heading-xl"><%= t("flex.tasks.index.title") %></h1>
+    <nav class="display-flex border-bottom-1px border-base-light">
+      <% 
+        filter_params = params.permit(:filter_date, :filter_type, :filter_status)
+        [
+          { 
+            name: t("flex.tasks.index.tabs.assigned"), 
+            path: url_for(filter_params.merge(filter_status: nil)), 
+            active: params[:filter_status] != "completed" 
+          },
+          { 
+            name: t("flex.tasks.index.tabs.completed"), 
+            path: url_for(filter_params.merge(filter_status: "completed")), 
+            active: params[:filter_status] == "completed" 
+          }
+        ].each do |tab| 
+      %>
+      <%= render partial: "strata/cases/nav_tab", locals: tab %>
+    <% end %>
+  </nav>
+  <!-- Filters -->
+  <div class="grid-row grid-gap-4 margin-bottom-4">
+    <div class="grid-col-12 tablet:grid-col-3">
+      <%= render partial: 'strata/tasks/type_filter', locals: { task_types: task_types } %>
+    </div>
+    <div class="grid-col-12 tablet:grid-col-3">
+      <%= render partial: 'strata/tasks/due_date_filter' %>
+    </div>
+    <div class="grid-col-12 tablet:grid-col-3">
+      <label class="usa-label" for="next-task-button" style="margin-bottom: 0.5rem">Actions:</label>
+      <%= button_to t("flex.tasks.actions.pick_next"), url_for(only_path: true, action: :pick_up_next_task), 
+                      id: "next-task-button",
+                      disabled: !unassigned_tasks&.any?,
+                      method: :post, 
+                      title: unassigned_tasks&.any? ? t("flex.tasks.messages.no_tasks_available") : nil,
+                      class: "usa-button" %>
+    </div>
+  </div>
+  <% if tasks&.any? %>
+    <!-- Tasks -->
+    <table class="usa-table usa-table--striped usa-table--sticky-header width-full">
+      <thead>
+        <tr>
+          <th data-sortable scope="col" role="columnheader" aria-sort="descending">
+            <%= t("flex.tasks.index.columns.col_due_date") %>
+          </th>
+          <th data-sortable scope="col" role="columnheader">
+            <%= t("flex.tasks.index.columns.col_type") %>
+          </th>
+          <th data-sortable scope="col" role="columnheader">
+            <%= t("flex.tasks.index.columns.col_case_id") %>
+          </th>
+          <th data-sortable scope="col" role="columnheader">
+            <%= t("flex.tasks.index.columns.col_created_date") %>
+          </th>
+        </tr>
+      </thead>
+      <tbody>
+        <%
+          tasks.each do |task|
+          %>
+        <tr>
+          <td data-sort-value="<%= time_since_epoch task.due_on %>">
+            <%= local_en_us task.due_on %>
+          </td>
+          <td data-sort-value="<%= task.type %>">
+            <%= link_to t("tasks.types.#{task.type.underscore}"), url_for(only_path: true, action: :show, id: task.id.to_s) %>
+          </td>
+          <td data-sort-value="<%= task.case_id %>"><%= task.case_id %></td>
+          <td data-sort-value="<%= time_since_epoch task.created_at %>">
+            <%= local_en_us task.created_at.to_date %>
+          </td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+  <div class="usa-sr-only usa-table__announcement-region" aria-live="polite"></div>
+<% else %>
+  <%= render partial: 'strata/tasks/no_tasks_alert' %>
+<% end %>
+</div>
+</div>
+</main>
diff --git a/app/views/strata/tasks/show.html.erb b/app/views/strata/tasks/show.html.erb
@@ -0,0 +1,67 @@
+<%# Show view for displaying a task
+    Required locals:
+    - assigned_user_display_text: String - The name of the user assigned to this task
+    - task: Flex::Task - The task object
+
+    Optional locals:
+    - task_details_partial: String - The HTML to render for the task details. If not provided, the view will look for a partial named after the task type within the `details` directory.
+    
+    Example usage:
+      render template: 'strata/tasks/default/show', locals: {
+        task: @task,
+        assigned_user_display_text: assigned_user&.full_name || "Not Assigned"
+      }
+%>
+<main class="usa-section">
+  <div class="grid-container">
+    <%
+      breadcrumbs = local_assigns[:breadcrumbs] || [
+        {
+          text: t("flex.tasks.breadcrumbs.home"),
+          link: "/"
+        },
+        {
+          text: t("flex.tasks.breadcrumbs.tasks"),
+          link: url_for(only_path: true, action: :index)
+        },
+        {
+          text: t("tasks.types.#{task.type.underscore}")
+        }
+      ]
+    %>
+    <%= render partial: "strata/shared/breadcrumbs", locals: { breadcrumbs: } %>
+
+    <h1 class="section-heading-h1"><%= t("tasks.types.#{task.type.underscore}") %></h1>
+    <div class="border-bottom-1px margin-bottom-1 padding-bottom-1">
+      <div class="grid-container">
+        <div class="grid-row grid-gap-2">
+          <div class="grid-col-auto">
+            <strong><%= t("flex.tasks.show.details.status") %>:</strong> <%= t("tasks.statuses.#{task.status}") %>
+          </div>
+          <div class="grid-col-auto">
+            <strong><%= t("flex.tasks.show.details.due_on") %>:</strong> <%= local_en_us task.due_on %>
+          </div>
+          <div class="grid-col-auto">
+            <strong><%= t("flex.tasks.show.details.assigned_to") %>:</strong> <%= assigned_user_display_text %>
+          </div>
+        </div>
+      </div>
+    </div>
+    <section class="usa-section padding-top-2">
+      <% if flash["task-message"] %>
+        <div class="usa-alert usa-alert--info usa-alert--no-icon">
+          <div class="usa-alert__body">
+            <p class="usa-alert__text">
+              <%= flash["task-message"] %>
+            </p>
+          </div>
+        </div>
+      <% end %>
+      <h2 class="section-heading-h2"><%= t("flex.tasks.show.headers.task_details") %></h2>
+      <% if content_for?(:task_details_partial) %>
+        <%= yield(:task_details_partial) %>
+      <% else %>
+        <%= render partial: "details/#{task.type.underscore}", locals: local_assigns.to_h %>
+      <% end %>
+    </section>
+  </main>
diff --git a/lib/generators/flex/staff/USAGE b/lib/generators/flex/staff/USAGE
@@ -9,8 +9,8 @@ Examples:
 
 This will create:
     Controllers:
-        app/controllers/staff_controller.rb - Main staff dashboard controller inheriting from Flex::StaffController
-        app/controllers/tasks_controller.rb - Tasks management controller inheriting from Flex::TasksController
+        app/controllers/staff_controller.rb - Main staff dashboard controller inheriting from Strata::StaffController
+        app/controllers/tasks_controller.rb - Tasks management controller inheriting from Strata::TasksController
 
     Views:
         app/views/staff/index.html.erb - Staff dashboard index page
diff --git a/lib/generators/flex/staff/templates/staff_controller.rb b/lib/generators/flex/staff/templates/staff_controller.rb
@@ -1,5 +1,5 @@
-class StaffController < Flex::StaffController
-  # This controller inherits from Flex::StaffController and provides access to the staff dashboard.
+class StaffController < Strata::StaffController
+  # This controller inherits from Strata::StaffController and provides access to the staff dashboard.
   before_action :authenticate_user!
 
   # TODO implement staff policy
diff --git a/lib/generators/flex/staff/templates/tasks_controller.rb b/lib/generators/flex/staff/templates/tasks_controller.rb
@@ -1,2 +1,2 @@
-class TasksController < Flex::TasksController
+class TasksController < Strata::TasksController
 end
diff --git a/spec/controllers/strata/tasks_controller_spec.rb b/spec/controllers/strata/tasks_controller_spec.rb
@@ -1,14 +1,14 @@
 require 'rails_helper'
 
-RSpec.describe Flex::TasksController, type: :controller do
+RSpec.describe Strata::TasksController, type: :controller do
   let(:user) { create(:user) }
   let(:application_form) { create(:test_application_form) }
   let(:case_record) { create(:test_case, application_form_id: application_form.id) }
   let(:task) { case_record.create_task(TestTask, case: case_record) }
 
   before do
     Rails.application.routes.draw do
-      namespace :flex do
+      namespace :strata do
         resources :tasks, only: [ :index, :show, :update ]
       end
     end
diff --git a/spec/dummy/app/controllers/staff_controller.rb b/spec/dummy/app/controllers/staff_controller.rb
diff --git a/spec/dummy/app/controllers/tasks_controller.rb b/spec/dummy/app/controllers/tasks_controller.rb
diff --git a/spec/dummy/app/views/staff/index.html.erb b/spec/dummy/app/views/staff/index.html.erb
diff --git a/spec/dummy/app/views/tasks/show.html.erb b/spec/dummy/app/views/tasks/show.html.erb
diff --git a/spec/helpers/strata/date_helper_spec.rb b/spec/helpers/strata/date_helper_spec.rb

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-module Flex`
	`1`	`+module Strata`
`2`	`2`	`# Controller for managing Flex::Task records. Handles listing, filtering, showing, and updating tasks.`
`3`	`3`	`# This controller helps a parent application manage tasks by not forcing the parent application to implement the same functionality.`
`4`	`4`	`class TasksController < ::StaffController`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@`
`29`	`29`	`}`
`30`	`30`	`]`
`31`	`31`	`%>`
`32`		`- <%= render partial: "flex/shared/breadcrumbs", locals: { breadcrumbs: } %>`
	`32`	`+ <%= render partial: "strata/shared/breadcrumbs", locals: { breadcrumbs: } %>`
`33`	`33`
`34`	`34`	`<h1 class="section-heading-h1"><%= t("tasks.types.#{task.type.underscore}") %></h1>`
`35`	`35`	`<div class="border-bottom-1px margin-bottom-1 padding-bottom-1">`