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

Allow multiple roles per admin user #1032

Closed
airbone42 opened this issue Feb 9, 2015 · 4 comments
Closed

Allow multiple roles per admin user #1032

airbone42 opened this issue Feb 9, 2015 · 4 comments
Assignees
Labels
feature request Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@airbone42
Copy link

Currently it's only possible to have one role per admin user, which is quite difficult to handle if you have multiple users which have overlapping roles.

E.g. we have usually Editors and Sales people. So we need two roles. Besides that we also have people who're allowed to do both, so we already have to maintain three roles which have a very big overlap in the ressources. We always have to update all the roles if something in the ressources changes. This problem gets even bigger if you have three or four normal roles which have to be mixed up.

Also the ressources which everyone should have, like the section to change the own password has to be added to every role individually instead of just having a "my account" role which you have to maintain only once and add it to every user.

@tzyganu
Copy link
Contributor

tzyganu commented Feb 9, 2015

👍

@ilol
Copy link

ilol commented Feb 19, 2015

Thanks, @airbone42. Great idea, and we even have registered task (MAGETWO-11357) in our product backlog related to this topic. But that is expensive feature.

I would discuss other side of this problem. What do you think about connection of our user role with Active Directory (FreeIPA) group? This would be helpful for user administrators to setup and sync Magento permissions with permissions to other system resources of merchant`s infrastructure.

@ilol ilol added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Feb 19, 2015
@airbone42
Copy link
Author

Hi @ilol,

a sync to AD is definitely a great feature but I would see it separated from this one. AD sync might be interesting for enterprise systems, but having shared roles is required by almost every merchant that has more than two users. We would even need that as an agency for our different set of supporting users, which btw are usually not registered in the AD of the merchants. ;)

@sshrewz sshrewz added CS and removed PS labels Jul 15, 2015
mmansoor-magento pushed a commit to magento-mpi/magento2 that referenced this issue May 9, 2016
@vkorotun vkorotun removed the CS label Aug 4, 2016
@piotrekkaminski
Copy link
Contributor

Thank you for your submission.

We recently made some changes to the way we process GitHub submissions to more quickly identify and respond to core code issues.

Feature Requests and Improvements should now be submitted to the new Magento 2 Feature Requests and Improvements forum (see details here).

We are closing this GitHub ticket and have moved your request to the new forum.

magento-team pushed a commit that referenced this issue Apr 20, 2017
magento-team pushed a commit that referenced this issue Jan 16, 2018
…gnores clicks if the content is empty. #1032

 - Merge Pull Request magento-engcom/magento2ce#1032 from RomaKis/magento2:8114
 - Merged commits:
   1. 5bac54f
   2. 4456872
   3. 42ee7cc
magento-team pushed a commit that referenced this issue Jan 16, 2018
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this issue Jun 22, 2018
MSI-950: [Configuration-Stocks-Delete Stock] Stock can not be deleted…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

7 participants