-
Notifications
You must be signed in to change notification settings - Fork 0
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
Flag licence for 2pt supplementary through recalculating bills #1302
Conversation
https://eaflood.atlassian.net/browse/WATER-4589 This PR is for rebuilding the recalculating bills pages for marking a licence as two-part tariff supplementary. Due to an extra page being added to the journey, the decision was made to rebuild the existing pages rather than try and insert a page from our system repo.
d4190ce
to
73eae2a
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.
Just a few little things
test/presenters/licences/supplementary/mark-for-supplementary-billing.presenter.test.js
Outdated
Show resolved
Hide resolved
app/services/licences/supplementary/submit-mark-for-supplementary-billing.service.js
Show resolved
Hide resolved
app/services/licences/supplementary/submit-mark-for-supplementary-billing.service.js
Show resolved
Hide resolved
Question for @Cruikshanks The two-part tariff supplementary bill runs journey is now hidden behind a feature flag. Does the 'recalculate link' need to be the same? The functionality all works and the existing supplementary flagging all works, it's just in a different place and the users obviously can't run a 2pt supp bill run. |
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.
https://eaflood.atlassian.net/browse/WATER-4589
This PR is for rebuilding the recalculating bill's pages for marking a licence as a two-part tariff supplementary. Due to an extra page being added to the journey, the decision was made to rebuild the existing pages rather than try and insert a page from our system repo.