Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Update observers-bp.md #1700

Merged
merged 2 commits into from
Feb 9, 2018
Merged

Update observers-bp.md #1700

merged 2 commits into from
Feb 9, 2018

Conversation

luckyduck
Copy link
Contributor

Make sure the adminhtml option is listed as well. Already described somewhere else, but showing all options on the point of use avoids further research.

luckyduck and others added 2 commits February 9, 2018 11:50
Make sure the adminhtml option is listed as well. Already described somewhere else, but showing all options on the point of use avoids further research.
@jeff-matthews jeff-matthews self-assigned this Feb 9, 2018
@jeff-matthews jeff-matthews self-requested a review February 9, 2018 15:41
@jeff-matthews jeff-matthews added 2.x Major Update Significant original updates to existing content labels Feb 9, 2018
@jeff-matthews
Copy link
Contributor

jeff-matthews commented Feb 9, 2018

Thanks for the contribution @luckyduck!

whatsnew
Listed the adminhtml option in the Observers best practices topic.

@jeff-matthews jeff-matthews merged commit 78710f6 into magento:develop Feb 9, 2018
@hostep
Copy link
Contributor

hostep commented Feb 9, 2018

Guys, don't forget that there are other area's as well which are being used on the frontend, see this issue which still hasn't been addressed yet: #908 ;)

@jeff-matthews
Copy link
Contributor

Thanks for the reminder @hostep. We're doing our best to address issues. Sometimes, a PR is easier for us to process given our current time and resource constraints.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Major Update Significant original updates to existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants