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

refactor: remove ecToken from modalContent request #1091

Merged
merged 1 commit into from May 14, 2024

Conversation

yeul
Copy link
Contributor

@yeul yeul commented May 10, 2024

Description

  • Removes ec_token from modalContent request to mitigate downstream calculation disparities when entering application flow of Pay Monthly.

Screenshots

Testing instructions

  • When typing in the Pay Monthly calculator, ensure ec_token is not being sent as a query param as part of the modalContent request.
  • ec_token should continue being sent to /credit-presentment/smart/modal endpoint.

@perco12 perco12 merged commit 0beb8d6 into paypal:develop May 14, 2024
51 checks passed
github-actions bot pushed a commit that referenced this pull request May 14, 2024
### [1.64.3](v1.64.2...v1.64.3) (2024-05-14)

### Bug Fixes

* origami styling ([#1080](#1080)) ([28d5fc1](28d5fc1))

### Code Refactoring

* ensure content attribute on meta element is not incorrectly set to undefined after closing modal ([#1083](#1083)) ([905905e](905905e))
* fix donut dash position ([#1089](#1089)) ([b2429be](b2429be))
* pass modal zoid polyfill duration as string ([#1066](#1066)) ([f1b66cd](f1b66cd))
* remove ecToken from modalContent request ([#1091](#1091)) ([0beb8d6](0beb8d6))
* remove extra older zalgo-promise dependency ([#1093](#1093)) ([3733916](3733916))
* update GB legal disclosure content in modals ([#1092](#1092)) ([78e759d](78e759d))
Copy link

🎉 This PR is included in version 1.64.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants