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

[TASK] Listen to ProcessOrderCheckStockEvent #139

Conversation

rintisch
Copy link
Collaborator

Allow to check the stock during the order process
by listening to the extended ProcessOrderCheckStockEvent.

Fixes #138

@rintisch rintisch force-pushed the task/issue-138-listen-to-processOrderCheckStockEvent branch from 003a9cb to 51a0ec7 Compare April 18, 2024 04:37
Allow to check the stock during the order process
by listening to the extended `ProcessOrderCheckStockEvent`.

Fixes extcode#138
@rintisch rintisch force-pushed the task/issue-138-listen-to-processOrderCheckStockEvent branch from 51a0ec7 to f4b439a Compare April 20, 2024 14:22
@rintisch
Copy link
Collaborator Author

Hint: The CI fails because some of the called methods are not present in the referenced branch of EXT:cart...

@extcode
Copy link
Owner

extcode commented May 30, 2024

Can you update the code to the renamed methods?

@rintisch rintisch force-pushed the task/issue-138-listen-to-processOrderCheckStockEvent branch from 68423a1 to a336182 Compare May 30, 2024 16:40
@rintisch
Copy link
Collaborator Author

rintisch commented May 30, 2024

@extcode I renamed the methods and made a force push.

For BE variants the variants of the cartProducts
needs to be compared with the products itself.
@rintisch rintisch force-pushed the task/issue-138-listen-to-processOrderCheckStockEvent branch from a336182 to 6d49b2a Compare May 30, 2024 16:42
@extcode extcode deleted the branch extcode:5.x-dev June 21, 2024 21:25
@extcode extcode closed this Jun 21, 2024
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