-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Improvements on event observer topic #8950
Improvements on event observer topic #8950
Conversation
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.
Good addition
@BarnyShergold Updated the PR with the suggested changes. Thanks |
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.
@ajithkumar-maragathavel A few editorial comments. The 3rd column of the table isn't necessary since 6 out 7 descriptions vary by one word.
@keharper Thanks for the suggestions. PR updated. |
@magento import code to magento-devdocs/devdocs |
@keharper the branch with code successfully imported into |
Hi @ajithkumar-maragathavel, thank you for your contribution! |
Purpose of this pull request
This pull request (PR) includes the below improvements:
Affected DevDocs pages
Links to Magento source code
whatsnew
Added an example and table to the Events and observers.