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

Show user's projects in user list view #623

Merged
merged 3 commits into from
Nov 11, 2020
Merged

Show user's projects in user list view #623

merged 3 commits into from
Nov 11, 2020

Conversation

landonreed
Copy link
Contributor

@landonreed landonreed commented Nov 11, 2020

Checklist

  • Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
  • Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
  • The description lists all applicable issues this PR seeks to resolve
  • The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

Fixes #622 by adding labels to user rows to indicate project permissions. Change is visible at https://gtfs-dev.ibi-transit.com/admin/users. The cases handled now are:

  • application admin
  • organization admin (mostly unused)
  • 🆕 list of projects user has (does not distinguish between admin/custom permissions)
  • 🆕 indication that user has no projects
  • 🆕 indication that user has projects that are missing from the database (either deleted or belonging to another Data Tools instance)

show-user-permissions-

@evansiroky evansiroky removed their assignment Nov 11, 2020
@landonreed landonreed changed the title Show user projects in list view Show user's projects in user list view Nov 11, 2020
@landonreed landonreed merged commit f58878c into dev Nov 11, 2020
@landonreed landonreed deleted the show-user-projects branch November 11, 2020 21:20
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.

Difficult to determine user permissions in list view
3 participants