Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merton][WW] Allow transfer of ggw subscription #5232

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MorayMySoc
Copy link
Contributor

@MorayMySoc MorayMySoc commented Oct 25, 2024

When a Green Garden Waste subscriber in Merton moves to another address in Merton, they can transfer the remainder of their ggw subscription to the new address

  1. As long as the new address does not already have a sub
  2. As long as there is a ggw on their previous property, and it is not in the renewal period
  3. They have brought their bin(s) with them

This is a staff only feature.

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

[skip changelog]

Copy link

codecov bot commented Oct 25, 2024

Codecov Report

Attention: Patch coverage is 97.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 83.07%. Comparing base (7296c20) to head (731640f).
Report is 33 commits behind head on master.

Files with missing lines Patch % Lines
perllib/FixMyStreet/App/Controller/Waste.pm 95.55% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5232      +/-   ##
==========================================
+ Coverage   82.36%   83.07%   +0.70%     
==========================================
  Files         409      410       +1     
  Lines       31999    34313    +2314     
  Branches     5095     5604     +509     
==========================================
+ Hits        26356    28504    +2148     
- Misses       4136     4257     +121     
- Partials     1507     1552      +45     

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

@MorayMySoc MorayMySoc removed the request for review from dracos October 25, 2024 12:00
Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good :) I don't think any of my comments are major, and I think the end date should hook up fine for the new sub.

perllib/FixMyStreet/App/Controller/Waste.pm Outdated Show resolved Hide resolved
templates/web/base/waste/bin_days_sidebar.html Outdated Show resolved Hide resolved
perllib/FixMyStreet/App/Form/Waste/Garden/Transfer.pm Outdated Show resolved Hide resolved
perllib/FixMyStreet/App/Form/Waste/Garden/Transfer.pm Outdated Show resolved Hide resolved
templates/web/base/waste/garden/transferred.html Outdated Show resolved Hide resolved
t/app/controller/waste_merton_garden.t Outdated Show resolved Hide resolved
t/app/controller/waste_merton_garden.t Outdated Show resolved Hide resolved
t/app/controller/waste_merton_garden.t Outdated Show resolved Hide resolved
perllib/FixMyStreet/App/Controller/Waste.pm Outdated Show resolved Hide resolved
perllib/FixMyStreet/App/Controller/Waste.pm Outdated Show resolved Hide resolved
Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, think you've covered everything I asked - just a couple of knock-on things from that.

perllib/FixMyStreet/Cobrand/Merton/Waste.pm Outdated Show resolved Hide resolved
Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost, but... the childdata might be a hash or an array, I'm afraid! See the double loop for the same type of data at https://github.com/mysociety/fixmystreet/blob/master/perllib/FixMyStreet/Roles/Cobrand/Echo.pm#L415

When a Green Garden Waste subscriber in Merton moves
to another address in Merton, they can transfer the
remainder of their ggw subscription to the new address

1) As long as the new address does not already have a sub
2) As long as there is a ggw on their previous property,
and it is not in the renewal period
3) They have brought their bin(s) with them

This is a staff only feature.

mysociety/societyworks#4551
@MorayMySoc
Copy link
Contributor Author

open311 changes also in 4551-merton-ggw-transfers on servers

Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

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