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

feat: Update DE Pi30 and Ratenzahlung modals to have EN variants #1076

Merged

Conversation

perco12
Copy link
Contributor

@perco12 perco12 commented Apr 17, 2024

Description

Update DE Modal Pi30 and Paypal Ratenzahlung to have en variants.

Screenshots

Testing instructions

Testing Links will be on the other PRs

@perco12 perco12 changed the title Added EN localization for DE Modal feat: Update DE Pi30 and Ratenzahlung modals to have EN variants Apr 18, 2024
@surekhaw surekhaw added snapshots Trigger new snapshots to be generated and removed snapshots Trigger new snapshots to be generated labels Apr 18, 2024
@perco12 perco12 added snapshots Trigger new snapshots to be generated and removed snapshots Trigger new snapshots to be generated labels Apr 18, 2024
@surekhaw surekhaw added the snapshots Trigger new snapshots to be generated label Apr 19, 2024
@surekhaw surekhaw removed the snapshots Trigger new snapshots to be generated label Apr 19, 2024
content/modals/DE/long_term.json Outdated Show resolved Hide resolved
content/modals/DE/long_term_en.json Outdated Show resolved Hide resolved
content/modals/DE/pay_in_1_en.json Show resolved Hide resolved
content/modals/DE/pay_in_1_en.json Outdated Show resolved Hide resolved
content/modals/DE/long_term_en.json Outdated Show resolved Hide resolved
tests/functional/v2/config/DE/DEV_DE_LONG_TERM_EN.js Outdated Show resolved Hide resolved
tests/functional/v2/config/DE/DEV_DE_LONG_TERM_EN_0APR.js Outdated Show resolved Hide resolved
@surekhaw surekhaw added the snapshots Trigger new snapshots to be generated label Apr 22, 2024
@@ -22,6 +22,12 @@ const LOCALE_CONFIG = config[LOCALE];
const ACCOUNT_CONFIG = LOCALE_CONFIG[ACCOUNT];
const integration = 'standalone';
const testFileName = 'longTerm';
const ALL_DE_ACCOUNTS = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add ALL_DE_ACCOUNTS and respective updates to all spec/snapshot-tests subfolders (not just standalone)

@surekhaw surekhaw removed the snapshots Trigger new snapshots to be generated label Apr 22, 2024
@surekhaw surekhaw added the snapshots Trigger new snapshots to be generated label Apr 22, 2024
error => console.error(error) // eslint-disable-line no-console
error => {
console.error(error); // eslint-disable-line no-console
return [0, 0];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be necessary. Adding en variants to other test files would have fixed timeout error occurring with previous attempts to update.

@surekhaw surekhaw removed the snapshots Trigger new snapshots to be generated label Apr 22, 2024
@danzhaaspaypal danzhaaspaypal merged commit c428184 into paypal:develop Apr 30, 2024
51 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 30, 2024
## [1.64.0](v1.63.0...v1.64.0) (2024-04-30)

### Features

* purchase protection messsage ([#1075](#1075)) ([c90c280](c90c280))
* Update DE Pi30 and Ratenzahlung modals to have EN variants ([#1076](#1076)) ([c428184](c428184))

### Code Refactoring

* allow mod XO learn more modal inside iframe ([#1079](#1079)) ([5884791](5884791))
* font feature ([#1061](#1061)) ([282c7fe](282c7fe))
Copy link

🎉 This PR is included in version 1.64.0 🎉

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