-
Notifications
You must be signed in to change notification settings - Fork 29
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
Use product and ratePlan qs params over amounts for support epic #12403
base: main
Are you sure you want to change the base?
Conversation
Size Change: +265 B (+0.03%) Total Size: 938 kB
ℹ️ View Unchanged
|
f6d4979
to
6b626e1
Compare
6b626e1
to
ae37cc4
Compare
ae37cc4
to
961712f
Compare
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
…otcom-rendering into use-product-for-selected-tier
…product-for-selected-tier
Currently the banner links to the landing page with e.g. |
@tomrf1 this currently works, but we want to remove those qs parameters as it just adds another level of transformation and complexity. This PR adds the functionality of being able to use the |
@@ -139,6 +137,9 @@ export const design: ConfigurableDesign = { | |||
filledProgress: stringToHexColour('052962'), | |||
progressBarBackground: stringToHexColour('cccccc'), | |||
goalMarker: stringToHexColour('000000'), | |||
goalColour: stringToHexColour('000000'), | |||
headlineColour: stringToHexColour('000000'), | |||
totalColour: stringToHexColour('000000'), |
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.
These are new properties needed for the version bump - assuming this should have been a breaking change but wasn't.
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.
The related change is blocked by a source upgrade, but I think your update here is fine
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.
looks good!
What does this change?
Uses the
product
andratePlan
values on the links through to support.theguardian.com.Updates
guardian/support-dotcom-components
(additive and patch, so won't have any backwards compatibility issues)Why?
On
support-frontend
we are moving away from using multiple layers of transformations, especially from amount => product, but rather using product directly, and trying to maintain the model and values as close to the Product API as possible.The URLs are now as follows (with tracking params omitted for clarity)
/us/checkout?product=Contribution&ratePlan=Monthly
/us/checkout?product=SupporterPlus&ratePlan=Monthly
/us/contribute/checkout?selected-contribution-type=ONE_OFF
/uk/contribution?product=Contribution&ratePlan=Monthly
/uk/contribution?product=SupporterPlus&ratePlan=Monthly
/uk/contribute
Depends on
Non-US
Screen.Recording.2024-09-19.at.11.26.12.mov
US
Screen.Recording.2024-09-19.at.11.36.43.mov