Skip to content

feat(next): add mapper function for sp3 to sp2 conversion#18279

Merged
StaberindeZA merged 1 commit intomainfrom
fxa-10967-rollout-map-func
Jan 24, 2025
Merged

feat(next): add mapper function for sp3 to sp2 conversion#18279
StaberindeZA merged 1 commit intomainfrom
fxa-10967-rollout-map-func

Conversation

@StaberindeZA
Copy link
Copy Markdown
Contributor

Because

  • The payments-next landing page needs to be able to map SP3 parameters to SP2 product and price IDs.

This pull request

  • Adds new mapper function to map offeringId+interval+currency to productId+priceId.
  • Adds config to provide correct values to mapping function

Issue that this pull request solves

Closes: #FXA-10967

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

@StaberindeZA StaberindeZA requested a review from a team as a code owner January 23, 2025 20:37
@StaberindeZA StaberindeZA force-pushed the fxa-10967-rollout-map-func branch from a136e62 to 52df7e2 Compare January 23, 2025 20:57
Comment thread libs/payments/ui/src/lib/nestapp/nextjs-actions.service.ts Outdated
Because:

- The payments-next landing page needs to be able to map SP3 parameters
  to SP2 product and price IDs.

This commit:

- Adds new mapper function to map offeringId+interval+currency to
  productId+priceId.
- Adds config to provide correct values to mapping function

Closes #FXA-10967
@StaberindeZA StaberindeZA force-pushed the fxa-10967-rollout-map-func branch from 52df7e2 to 619dc52 Compare January 24, 2025 15:20
@StaberindeZA StaberindeZA merged commit 5f781ad into main Jan 24, 2025
@StaberindeZA StaberindeZA deleted the fxa-10967-rollout-map-func branch January 24, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants