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

UI identity lists #4655

Merged
merged 7 commits into from
May 30, 2018
Merged

UI identity lists #4655

merged 7 commits into from
May 30, 2018

Conversation

meirish
Copy link
Contributor

@meirish meirish commented May 30, 2018

This uses the new key_info block on identity list endpoints to make the Identity UI (Access > Entities and Access > Groups) nicer to navigate and reason about. Groups now also show a "Parent Groups" tab that lists the groups a group is a member of.

screen shot 2018-05-29 at 9 19 43 pm
screen shot 2018-05-29 at 9 20 21 pm
screen shot 2018-05-29 at 9 20 29 pm
screen shot 2018-05-29 at 9 21 07 pm

Also makes a visual adjustment to the tabs subnav to be closer to the designs.

@meirish meirish requested review from joshuaogle and a team May 30, 2018 02:26
@meirish
Copy link
Contributor Author

meirish commented May 30, 2018

screen shot 2018-05-29 at 9 51 31 pm

Removed the side padding on the tabs so the spacing is nicer.

@meirish meirish added this to the 0.10.2 milestone May 30, 2018
@meirish meirish merged commit 6150120 into master May 30, 2018
@meirish meirish deleted the ui-identity-lists branch May 30, 2018 02:56
Copy link
Contributor

@joshuaogle joshuaogle left a comment

Choose a reason for hiding this comment

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

Had a couple of small comments, but no blockers. Looks great! Thanks for squeezing this in

{{#if item.aliases.length}}
{{pluralize item.aliases.length "alias"}}
{{/if}}
{{else}}
Copy link
Contributor

Choose a reason for hiding this comment

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

✂️ ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh left that in from groups, can take out the outer if entirely until we land other groups stuff.

<div class="has-text-grey">
{{item.id}}
</div>
<span class="tag"> {{item.mountType}} </span>
Copy link
Contributor

Choose a reason for hiding this comment

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

Did this need spaces between for a reason?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lol nope, was being hasty - will fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants