Skip to content
This repository has been archived by the owner on Jan 3, 2022. It is now read-only.

Permission improvements #27

Merged
merged 2 commits into from
Jun 4, 2021
Merged

Permission improvements #27

merged 2 commits into from
Jun 4, 2021

Conversation

jozuas
Copy link
Contributor

@jozuas jozuas commented Jun 3, 2021

  • Reduced the number of permissions granted to what is necessary
  • Improved the CLI interface for admin-setup and add-user
    1. Opt-out instead of opt-in for full s3 access. Most of the current users just want to get going, advanced s3 permissions are an exception rather than the rule.
    2. I noticed that it is more convenient to have profile first and then username last when adding multiple users.

@jozuas jozuas requested a review from seuqaj114 June 3, 2021 15:25
@jozuas jozuas merged commit c5dafeb into master Jun 4, 2021
@jozuas jozuas deleted the permission-improvements branch June 6, 2021 16:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant