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.

Sign up for GitHub

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

Jump to bottom

fix: trustpay eps redirection in cypress #6529

Merged
merged 1 commit into from
Nov 12, 2024
Merged

fix: trustpay eps redirection in cypress #6529

merged 1 commit into from
Nov 12, 2024

Conversation

pixincreate
Copy link
Member

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Fix EPS Bank Redirect redirection in Cypress.

Closes #6528

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

NIL

How did you test it?

image

Checklist

  • I formatted the code prettier . --write
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added A-CI-CD Area: Continuous Integration/Deployment S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Nov 11, 2024
@pixincreate pixincreate added this to the November 2024 Release milestone Nov 11, 2024
@pixincreate pixincreate self-assigned this Nov 11, 2024
@pixincreate pixincreate requested a review from a team as a code owner November 11, 2024 10:19
@semanticdiff-com SemanticDiff.com
Copy link

semanticdiff-com bot commented Nov 11, 2024 •
edited
Loading

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/cypress/support/redirectionHandler.js  27% smaller

@pixincreate
Copy link
Member Author

merging this pr to stop trustpay bank redirects from failing in tomorrow's deployment.

@pixincreate pixincreate added this pull request to the merge queue Nov 11, 2024
@pixincreate pixincreate removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Nov 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 11, 2024
@pixincreate pixincreate added this pull request to the merge queue Nov 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 12, 2024
@likhinbopanna likhinbopanna added this pull request to the merge queue Nov 12, 2024
Merged via the queue into main with commit 7f4f55b Nov 12, 2024
21 checks passed
@likhinbopanna likhinbopanna deleted the fix-trustpay-eps-redirection branch November 12, 2024 07:17
bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
@pixincreate
fix: trustpay eps redirection in cypress (#6529)
7f71cec
bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
@pixincreate
fix: trustpay eps redirection in cypress (#6529)
5d47198
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@likhinbopanna likhinbopanna likhinbopanna approved these changes

@Gnanasundari24 Gnanasundari24 Gnanasundari24 approved these changes

Assignees

@pixincreate pixincreate

Labels
A-CI-CD Area: Continuous Integration/Deployment
Projects
None yet
Milestone
November 2024 Release
Development

Successfully merging this pull request may close these issues.

fix: trustpay eps redirection in cypress
3 participants
@pixincreate @Gnanasundari24 @likhinbopanna