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

[ECP-8778] Create new state for adyen_authorized status #2375

Merged
merged 4 commits into from
Nov 29, 2023
Merged

Conversation

candemiralp
Copy link
Member

Description

A merchant can select he order status for order creation on the configuration page. Since, adyen_authorized has been assigned to state new after implementing #2316, merchant can select Authorized status for order creation, which is not always true.

In order to tackle this issue, a new state was generated and adyen_authorized status was assigned to this new state.

Tested scenarios

  • Running migration script
  • Manual capture payments

@candemiralp candemiralp requested a review from a team as a code owner November 29, 2023 09:54
Copy link

sonarcloud bot commented Nov 29, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

83.3% 83.3% Coverage
0.0% 0.0% Duplication

@candemiralp candemiralp merged commit 74d15b4 into develop Nov 29, 2023
7 checks passed
@candemiralp candemiralp deleted the ECP-8778 branch November 29, 2023 12:40
@candemiralp candemiralp added the Fix Indicates a bug fix label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Indicates a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants