-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Magefan succeeded to add their own entry #22
Comments
I'll work on it later :) unless you already have a fix |
Hey @thomas-kl1 Thanks for reporting this. Possibly they add this menu item via plugin or observer instead of menu.xml. In case you didn’t get a chance to look at this, I will be fixing this by tomorrow. |
Just pushed a fix 2e6abb4 not sure if it's optimal. Please give it a look and see if you can suggest changes. |
thanks @rav-redchamps for the fix! Nice to see it will also prevents any third party vendors to add menu entry with this method |
The text was updated successfully, but these errors were encountered: