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

[BP-1898] Invoice email sending is forced to be non-async #638

Open
igorwulff opened this issue Nov 22, 2022 · 1 comment
Open

[BP-1898] Invoice email sending is forced to be non-async #638

igorwulff opened this issue Nov 22, 2022 · 1 comment
Labels
Bug Bugs, functions that are not working like they should. Refactor This will be part of a refactoring process. Status: Waiting for release This issue has been solved, but is not released yet.

Comments

@igorwulff
Copy link

$this->invoiceSender->send($invoice, true);

I'm curious if this is behavior that we really want. Apparently invoice sending through this module is directly being send when a push from buckaroo happens .Why not control this behavior by settings defined by Magento in regards to async/sync sending of emails? Its causing some issues for one of our clients, since in parallel a lot of push calls were being fired to Magento, with slow email generation on our side because servers got overloaded by traffic next to these heavy calls.

@Buckaroo-Rens Buckaroo-Rens added Bug Bugs, functions that are not working like they should. Refactor This will be part of a refactoring process. labels Nov 23, 2022
@Buckaroo-Rens
Copy link
Collaborator

Hi @igorwulff thanks for issue. We are indeed aware of this and already in process of refactoring this part of our code. Not only this specific part but the whole push file/logic is going to be improved, standardized en more easy to read/use. Unfortunately i don't have an hotfix for this right now but it's high on our list.

Please also check #633 for some more info.

@Buckaroo-Rens Buckaroo-Rens changed the title Invoice email sending is forced to be non-async [BP-1898] Invoice email sending is forced to be non-async Nov 30, 2022
@Buckaroo-Rene Buckaroo-Rene added the Status: Waiting for release This issue has been solved, but is not released yet. label Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bugs, functions that are not working like they should. Refactor This will be part of a refactoring process. Status: Waiting for release This issue has been solved, but is not released yet.
Projects
None yet
Development

No branches or pull requests

3 participants