Move Assigned to Another Crew to bottom accordion, remove Schedule section #214

Closed
opened 2026-06-14 03:02:06 +00:00 by ldraney · 0 comments
Owner

Type

Feature

Lineage

Standalone — requested by Lucas during session.

Repo

ldraney/landscaping-assistant

User Story

As a crew lead
I want the Week view simplified to just Assign Properties with day links
So that I don't see duplicate schedule information and can navigate to daily queues from one place

Context

The Schedule accordion in the Week view duplicates what Assign Properties already shows — both display the same properties with the same day columns. The only useful piece in Schedule was the day-letter links (M/T/W/T/F/S/S) that navigate to each day's work queue. Those links should move to the Assign Properties header. Additionally, "Assigned to Another Crew" should be the bottom accordion on the Today work queue view.

File Targets

Files to modify:

  • app/views/work_queue_items/index.html.erb -- move other_crew_section render to bottom
  • app/views/weeks/index.html.erb -- remove Schedule accordion, add day links to Assign header
  • app/controllers/weeks_controller.rb -- remove move action, @assigned, @item_ids
  • app/assets/stylesheets/application.css -- remove week-grid/week-move CSS, add day-link styles
  • config/routes.rb -- remove move route
  • spec/requests/weeks_spec.rb -- remove move specs, update Schedule-dependent specs

Files to delete:

  • app/javascript/controllers/week_move_controller.js -- no longer used

Feature Flag

none

Acceptance Criteria

  • Today view: Assigned to Another Crew is the last accordion section
  • Week view: Schedule accordion is removed
  • Week view: Day letters in Assign Properties header link to the daily work queue
  • Today highlight works on day-letter links
  • All specs pass

Test Expectations

  • Existing specs updated to remove move/day-picker references
  • New spec: day links render with week-assign-day-link class
  • Run command: bundle exec rspec spec/requests/weeks_spec.rb

Constraints

  • Keep existing assign/toggle behavior untouched
  • Day-link styling should match the removed week-grid-day-link style

Checklist

  • PR opened
  • Tests pass
  • No unrelated changes
  • landscaping-assistant -- project this affects
### Type Feature ### Lineage Standalone — requested by Lucas during session. ### Repo `ldraney/landscaping-assistant` ### User Story As a crew lead I want the Week view simplified to just Assign Properties with day links So that I don't see duplicate schedule information and can navigate to daily queues from one place ### Context The Schedule accordion in the Week view duplicates what Assign Properties already shows — both display the same properties with the same day columns. The only useful piece in Schedule was the day-letter links (M/T/W/T/F/S/S) that navigate to each day's work queue. Those links should move to the Assign Properties header. Additionally, "Assigned to Another Crew" should be the bottom accordion on the Today work queue view. ### File Targets Files to modify: - `app/views/work_queue_items/index.html.erb` -- move other_crew_section render to bottom - `app/views/weeks/index.html.erb` -- remove Schedule accordion, add day links to Assign header - `app/controllers/weeks_controller.rb` -- remove move action, @assigned, @item_ids - `app/assets/stylesheets/application.css` -- remove week-grid/week-move CSS, add day-link styles - `config/routes.rb` -- remove move route - `spec/requests/weeks_spec.rb` -- remove move specs, update Schedule-dependent specs Files to delete: - `app/javascript/controllers/week_move_controller.js` -- no longer used ### Feature Flag none ### Acceptance Criteria - [ ] Today view: Assigned to Another Crew is the last accordion section - [ ] Week view: Schedule accordion is removed - [ ] Week view: Day letters in Assign Properties header link to the daily work queue - [ ] Today highlight works on day-letter links - [ ] All specs pass ### Test Expectations - [ ] Existing specs updated to remove move/day-picker references - [ ] New spec: day links render with `week-assign-day-link` class - Run command: `bundle exec rspec spec/requests/weeks_spec.rb` ### Constraints - Keep existing assign/toggle behavior untouched - Day-link styling should match the removed week-grid-day-link style ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes ### Related - `landscaping-assistant` -- project this affects
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/landscaping-assistant#214
No description provided.