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

Pricing summary refactor #1801

Closed
wants to merge 2 commits into from

Conversation

jaucourt
Copy link
Collaborator

Pricing summary refactor

https://eaflood.atlassian.net/browse/IWTF-3322

Refactor the pricing summary which is doing a lot of work with no caching

@jaucourt jaucourt added the enhancement New feature or request label Sep 25, 2023
@ScottDormand96
Copy link
Collaborator

I think it is easier to read with the changes you've made. Aware of risk with changing code but feel testing isn't hugely time consuming to check with all scenarios is displaying correctly. Happy to go ahead with changes if content pricing for all concessions is likely stay the same

@jaucourt jaucourt closed this Nov 15, 2023
@jaucourt jaucourt deleted the feature/iwtf-3322-pricing-summary-refactor branch November 15, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants