-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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(Mollie): Prevent duplicate payments #2691
Merged
michaelbromley
merged 17 commits into
vendure-ecommerce:minor
from
martijnvdbrug:feat/mollie-prevent-duplicate-payments
Mar 4, 2024
Merged
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
0cb2df8
feat(payments-plugin): Setup of preventing duplicate payments
martijnvdbrug 124f169
feat(payments-plugin): Accepted states
martijnvdbrug 9e58097
feat(payments-plugin): Mollie locale preservation #2270 and type fixes
martijnvdbrug 138d9ff
feat(core): Export order state machine
martijnvdbrug fb4cace
feat(payments-plugin): wip
martijnvdbrug 9c1df83
feat(payments-plugin): Live testing of duplicate payments
martijnvdbrug 9e51be2
feat(payments-plugin): Forgot to add files
martijnvdbrug 4642f9f
feat(payments-plugin): Reusing existing order if possible
martijnvdbrug 07d9b4d
feat(payments-plugin): Correct test naming
martijnvdbrug 1e65032
feat(payments-plugin): E2e test fixed
martijnvdbrug f7dcc31
feat(payments-plugin): Docs
martijnvdbrug 5a720bb
feat(payments-plugin): Cleaner usage of type
martijnvdbrug 9e3ca27
feat(payments-plugin): Updating existing order and lines
martijnvdbrug a31816f
feat(payments-plugin): Updating existing order and lines
martijnvdbrug cd7a39b
feat(payments-plugin): implementation finished
martijnvdbrug db75544
feat(payments-plugin): e2e finished
martijnvdbrug 3aecc7b
feat(payments-plugin): unused helper function
martijnvdbrug File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
feat(core): Export order state machine
- Loading branch information
commit 138d9fffd4c12080b774d1486a6e39d934020051
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this,
this.injector.get(OrderStateMachine);
in the Mollie service throws an error.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it throws an error if you use a deep import to this?
weird, but ok - it's worth exposing anyway.