-
Couldn't load subscription status.
- Fork 6
Fixes usermenu #207
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
Fixes usermenu #207
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR refactors the user menu route to use a Class diagram for updated MenuListener route handlingclassDiagram
class MenuListener {
+createMenu(factory, menu)
-route: string (changed from 'user_overview' to 'user_admin_overview')
}
MenuListener --|> DefaultMenuListener
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Was renamed in Framework-User-Implementation-Example
Js is already imported in head with assetmapper
Summary by Sourcery
Update user menu route prefixes and clean up template asset inclusions
Bug Fixes:
Enhancements:
Documentation: