-
Notifications
You must be signed in to change notification settings - Fork 11
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
[Task]: Follow up based on latest Pimcore 11 changes #134
Comments
what, if we just create two different JS files for that permission helper ... one for Pimcore 10 and one for Pimcore 11 and include them depending on the Pimcore version in use? Or do we also have a problem when using perspectives in Pimcore 11 that where created in Pimcore 10 ? |
AFAIK it would be more a visual and "cleanliness" problem, meaning that it would display "extra" stuff that might have no meaning nor effect, if the related bundle is not present/installed. Splitting the file sounds good to me. In other bundles, we are adding these perspective this way or so |
Thanks a lot for reporting the issue. We did not consider the issue as "Pimcore:Priority", "Pimcore:ToDo" or "Pimcore:Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding. |
Improvement description
Potentially we need to remove these perspectives from the list or have a bc layer for 10/11?
newsletter-bundle extracted
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Line 26 in 104ed6c
web-to-print-bundle
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Line 29 in 104ed6c
removal of import from server
https://github.com/pimcore/perspective-editor/blob/1.x/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js#L55
removal of import from url
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Line 56 in 104ed6c
See also [Task]: Fix docs about removal of import from server and url pimcore#15080
google-marketing-bundle
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Lines 111 to 114 in 104ed6c
reports
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Line 108 in 104ed6c
system-info
perspective-editor/src/Resources/public/js/pimcore/perspective/menuItemPermissionHelper.js
Lines 128 to 134 in 104ed6c
and so on...
The text was updated successfully, but these errors were encountered: