Skip to content

[Bromley] Add dispute missed collection enquiry#5825

Open
MorayMySoc wants to merge 1 commit intomasterfrom
5311-bromley-missed-dispute
Open

[Bromley] Add dispute missed collection enquiry#5825
MorayMySoc wants to merge 1 commit intomasterfrom
5311-bromley-missed-dispute

Conversation

@MorayMySoc
Copy link
Contributor

Add a flow for disputing a missed collection when containers marked as 'not presented'.

Give notices if contaminated or too late to dispute.

https://github.com/mysociety/societyworks/issues/5311

[skip changelog]

@MorayMySoc MorayMySoc requested a review from dracos January 27, 2026 15:14
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 82.50000% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.73%. Comparing base (7a5a17f) to head (84d25eb).

Files with missing lines Patch % Lines
perllib/FixMyStreet/Cobrand/Bromley.pm 79.41% 2 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5825      +/-   ##
==========================================
+ Coverage   82.70%   82.73%   +0.02%     
==========================================
  Files         458      459       +1     
  Lines       35885    35924      +39     
  Branches     5879     5890      +11     
==========================================
+ Hits        29679    29720      +41     
+ Misses       4491     4483       -8     
- Partials     1715     1721       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dracos dracos force-pushed the 5311-bromley-missed-dispute branch from b8c723e to ad582f7 Compare February 2, 2026 07:49
@dracos
Copy link
Member

dracos commented Feb 2, 2026

I've added a bunch of fixup commits, replacing the status flag with something reading the data it has as discussed, and moving to a subclass (still some code in the hook to decide which page to start with etc), and added the code to deal with the special extra data field on this event type - let me know if it all makes sense!

Add a flow for disputing a missed collection when containers
marked as 'not presented'.

Give notices if contaminated or too late to dispute.

We normally use the Echo resolution code to map to a response template,
but this event type is not using those, but instead setting an extra
data field to the outcome of the event.
@dracos dracos force-pushed the 5311-bromley-missed-dispute branch from ad582f7 to 84d25eb Compare February 6, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants