Skip to content

Embassy passports: show RECEIVED badge on My Plan when admin marks received#69

Merged
Kitkatnik merged 1 commit into
mainfrom
Kitkatnik/passport-received
May 1, 2026
Merged

Embassy passports: show RECEIVED badge on My Plan when admin marks received#69
Kitkatnik merged 1 commit into
mainfrom
Kitkatnik/passport-received

Conversation

@Kitkatnik
Copy link
Copy Markdown
Collaborator

Summary

  • The admin Mark Received button already set passport_received_at and removed the application from the active queue, but the user's My Plan card never read that timestamp — so it kept showing Ready (or Processing).
  • Adds a received branch above ready in the state cascade in _plan_item.html.erb so received takes precedence, plus a navy embassy-received-badge and casual "you're all set" copy. Cancel × is hidden once received.
  • New integration test in plan_test.rb sets both timestamps and asserts Received wins over Ready.

Test plan

  • bin/rails test — 361 runs, 0 failures
  • bin/rubocop, bin/brakeman, bin/bundler-audit, bin/importmap audit — all clean
  • Manual: as admin, hit Mark Received on a Ready application and confirm the user's /plan flips from green Ready to navy Received with cancel × hidden.

…ceived

The admin Mark Received button already set passport_received_at and removed
the application from the active queue, but the user's My Plan card never
read that timestamp, so it kept showing Ready (or Processing). Add a
"received" branch above "ready" in the state cascade so received takes
precedence, plus a navy badge and a casual "you're all set" copy block.
@railway-app
Copy link
Copy Markdown

railway-app Bot commented May 1, 2026

🚅 Deployed to the ruby-embassy-pr-69 environment in ruby-embassy

Service Status Web Updated (UTC)
ruby-embassy ✅ Success (View Logs) Web May 1, 2026 at 1:30 pm

@railway-app railway-app Bot temporarily deployed to ruby-embassy / ruby-embassy-pr-69 May 1, 2026 13:28 Destroyed
@Kitkatnik Kitkatnik merged commit fe55996 into main May 1, 2026
6 checks passed
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