-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
[DOCS] Generating invoice after payment #12887
Conversation
arti0090
commented
Aug 13, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
License | MIT |
bc33d46
to
88bc0ec
Compare
} | ||
} | ||
|
||
**2.** The ``NoInvoiceOnOrderPlacedListener`` will not do anything (what a lazy boy), but as we are changing the behavior, it still has very important role: |
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.
you can also remove this service from the container, but I am not sure if it will be a better or a different solution :)
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.
Well, that's true but I think this is the easier solution ( and much consistent with other cookbooks).
88bc0ec
to
68cb082
Compare
Co-authored-by: Grzegorz Sadowski <sadowskigp@gmail.com>
Thank you, @arti0090! 🥇 |