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] Initial bulky waste functionality #4924
base: merton-ww-initial
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## merton-ww-initial #4924 +/- ##
=====================================================
- Coverage 82.64% 82.63% -0.01%
=====================================================
Files 397 399 +2
Lines 30799 30895 +96
Branches 4861 4873 +12
=====================================================
+ Hits 25453 25530 +77
- Misses 3896 3904 +8
- Partials 1450 1461 +11 ☔ View full report in Codecov by Sentry. |
042f796
to
1e7efcd
Compare
81d1439
to
4f3d1f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good! Couple of comments but they are purely cosmetic 🙂
|
||
use_ok 'Integrations::Adelante'; | ||
|
||
my %sent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to be used anywhere, is it still needed?
wasteworks_config => { | ||
base_price => '6075', | ||
band1_price => '3700', | ||
band1_max => 3, | ||
items_per_collection_max => 6, | ||
per_item_costs => 0, | ||
show_location_page => 'users', | ||
item_list => [ | ||
{ bartec_id => '83', name => 'Bath' }, | ||
{ bartec_id => '84', name => 'Bathroom Cabinet /Shower Screen' }, | ||
{ bartec_id => '85', name => 'Bicycle' }, | ||
{ bartec_id => '3', name => 'BBQ' }, | ||
{ bartec_id => '6', name => 'Bookcase, Shelving Unit' }, | ||
], | ||
}, | ||
); | ||
$body->update; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indentation has gone slightly awry here.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
8dec216
to
ee495cf
Compare
Starts off by renaming the various payment Roles, moving those with no relevance to FixMyStreet under Integrations/Roles directly, and moving the cobrand-based payment roles under FixMyStreet/Roles/Cobrand/. Then adds a new payment integration provider (operates very similarly to the existing SCP one). Then adds the basic bulky waste code for Merton.
For https://github.com/mysociety/societyworks/issues/4270 [skip changelog]