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

Added config for success fail redirect path #780

Merged
merged 3 commits into from
Mar 25, 2025

Conversation

kevinmeijer97
Copy link
Member

Added some config to change the redirect path in case of a failed checkout.

@royduin
Copy link
Member

royduin commented Mar 25, 2025

Feedback from #774;

Maybe we should move all checkout configs to it's own file? @indykoning? Currently it's mixed:

And should we use named routes here?

And @indykoning his reaction:

That might make the config nicer, do we see any other configs being added for the checkout in the future?
If it's just the 2 this is fine, if it becomes more maybe we should do the separate config.

Merging for now, maybe something for #769 to change this config + maybe change this to named routes in the future?

@royduin royduin merged commit ca7ba9f into master Mar 25, 2025
12 checks passed
@royduin royduin deleted the feature/checkout-success-redirect branch March 25, 2025 07:37
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