Skip to content

HOLD: Titleize windows type short_names#1333

Open
maebeale wants to merge 1 commit intomainfrom
maebeale/titleize-windows-type-names
Open

HOLD: Titleize windows type short_names#1333
maebeale wants to merge 1 commit intomainfrom
maebeale/titleize-windows-type-names

Conversation

@maebeale
Copy link
Collaborator

@maebeale maebeale commented Mar 3, 2026

What is the goal of this PR and why is this important?

  • Windows type short_names were stored as all-caps (ADULT, CHILDREN, COMBINED) which looked harsh in the UI
  • Production data has already been updated to title case — this aligns the codebase to match

How did you approach the change?

  • Updated db/seeds.rb and db/seeds/dummy_dev_seeds.rb
  • Updated controller lookups that reference short_name by string (e.g. find_by(short_name: "Adult"))
  • Updated rake task (story_data.rake)
  • Updated factory traits (:adult, :children, :combined)
  • Updated view specs, request specs, system tests, and model spec expectations

UI Testing Checklist

  • Verify windows type names display as "Adult", "Children", "Combined" (not all-caps) across the app
  • Run seeds successfully: bin/rails db:seed
  • All specs pass: bundle exec rspec

Anything else to add?

  • No migration needed — production data was already updated directly
  • This is a codebase alignment change to match what's already in production

🤖 Generated with Claude Code

ADULT → Adult, CHILDREN → Children, COMBINED → Combined.
Updates seeds, factories, controller lookups, rake tasks, and specs
to match production data.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@maebeale maebeale force-pushed the maebeale/titleize-windows-type-names branch from c4c4d7c to fdf5208 Compare March 4, 2026 10:45
@maebeale maebeale changed the title Titleize windows type short_names HOLD: Titleize windows type short_names Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant