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

MM-18590 - Custom Admin Settings #18

Merged
merged 9 commits into from
Feb 4, 2020
Merged

MM-18590 - Custom Admin Settings #18

merged 9 commits into from
Feb 4, 2020

Conversation

marianunez
Copy link
Contributor

@marianunez marianunez commented Jan 23, 2020

Summary

Implemented custom admin settings to configure custom attribute that can have name with the associated users and/or groups that it would be applied to.

This first implementation of this UI allows for users to be searched and selected by username. Groups must be added by group id separated by a space (due to limitations by MM-22071)

Screenshots

Screen Shot 2020-01-29 at 9 23 17 PM

Screen Shot 2020-01-29 at 9 26 24 PM

Screen Shot 2020-01-29 at 9 27 58 PM

Ticket Link

MM-18590

@marianunez marianunez added the Work In Progress Not yet ready for review label Jan 23, 2020
@marianunez marianunez added 1: UX Review Requires review by a UX Designer 2: Dev Review Requires review by a core committer and removed Work In Progress Not yet ready for review labels Jan 28, 2020
@marianunez marianunez added the 3: QA Review Requires review by a QA tester label Jan 30, 2020
Copy link
Contributor

@lieut-data lieut-data left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! A few comments, but nothing strictly blocking.

@marianunez marianunez removed the 1: UX Review Requires review by a UX Designer label Feb 3, 2020
Copy link

@prapti prapti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected.

@prapti prapti added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Feb 4, 2020
@marianunez marianunez merged commit 1402746 into master Feb 4, 2020
@marianunez marianunez deleted the MM-18590 branch February 4, 2020 02:25
@jfrerich jfrerich mentioned this pull request Feb 6, 2020
@hanzei hanzei added the Docs/Needed Requires documentation label Mar 21, 2020
@aaronrothschild aaronrothschild removed the Docs/Needed Requires documentation label Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants