Skip to content
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

Review User/Manage menus with the aim to merge them in one #10819

Open
1 of 4 tasks
allyoucanmap opened this issue Feb 17, 2025 · 0 comments
Open
1 of 4 tasks

Review User/Manage menus with the aim to merge them in one #10819

allyoucanmap opened this issue Feb 17, 2025 · 0 comments
Assignees
Milestone

Comments

@allyoucanmap
Copy link
Contributor

allyoucanmap commented Feb 17, 2025

Description

We would like to merge the existing manager menu and login menu in a single dropdown:

  • admin user
    Image

  • normal user
    Image

Expected changes

  • deprecate and remove ManagerMenu from localConfing.json and jsdoc
  • Merge the logic from ManagerMenu inside the Login plugin allowing the injection of menu items from containers (see current MangerMenu logic)
  • Evaluate a refactor of Login dropdown component from class to function removing also unused logic (eg. unsaved changes because already managed by save plugin)

What kind of improvement you want to add? (check one with "x", remove the others)

  • Minor changes to existing features

Other useful information

@tdipisa tdipisa added this to the 2025.01.00 milestone Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants