Skip to content

Display group names in IAM tables#1219

Merged
david-crespo merged 12 commits into
mainfrom
user-group-names
Oct 10, 2022
Merged

Display group names in IAM tables#1219
david-crespo merged 12 commits into
mainfrom
user-group-names

Conversation

@david-crespo

@david-crespo david-crespo commented Oct 10, 2022

Copy link
Copy Markdown
Collaborator

Waiting on API support.

2022-10-10-group-badge-org-iam

2022-10-10-group-badge-project-iam

@vercel

vercel Bot commented Oct 10, 2022

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
console-ui-storybook ✅ Ready (Inspect) Visit Preview Oct 10, 2022 at 9:38PM (UTC)

@just-be-dev

Copy link
Copy Markdown
Contributor

Could we add a group to the mock data to show that case?

@david-crespo

Copy link
Copy Markdown
Collaborator Author

Working on it!

Comment thread libs/api/roles.ts
roleSource,
}))
}, [roleAssignments, roleSource, users])
}, [roleAssignments, roleSource, users, groups])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As ever this makes me wish the records and tuples proposal would pass so we could have some real immutability.

@just-be-dev just-be-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Whenever the API changes come through this looks good as a first pass. 👍

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