8000 feat: Implied 'member' roles for site and organization by Emyrk · Pull Request #1917 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

feat: Implied 'member' roles for site and organization #1917

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
Jun 1, 2022
Merged

feat: Implied 'member' roles for site and organization #1917

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
501e581
feat: Member roles are implied and never exlpicitly added
Emyrk May 31, 2022
51521ca
Rename "GetAllUserRoles" to "GetAuthorizationRoles"
Emyrk May 31, 2022
844c9ce
feat: Add migration to remove implied roles
Emyrk May 31, 2022
28e3957
Add comment to sql
Emyrk May 31, 2022
058e7ab
test: Update test
Emyrk May 31, 2022
9a3d38e
handle null string array
Emyrk May 31, 2022
5060443
Merge remote-tracking branch 'origin/main' into stevenmasley/implied_…
Emyrk May 31, 2022
6a89023
rename user auth role middleware
Emyrk Jun 1, 2022
ccecdda
fixup! rename user auth role middleware
Emyrk Jun 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
handle null string array
  • Loading branch information
@Emyrk
Emyrk committed May 31, 2022
commit 9a3d38e2b9e548f2efbf9619eba120108b777a7b
2 changes: 1 addition & 1 deletion coderd/organizations.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ func (api *API) postOrganizations(rw http.ResponseWriter, r *http.Request) {
},
})
if err != nil {
return xerrors.Errorf("create organization member: %w", err)
return xerrors.Errorf("create organization admin: %w", err)
}
return nil
})
Expand Down
2 changes: 1 addition & 1 deletion coderd/users.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -751,7 +751,7 @@ func (api *API) createAPIKey(rw http.ResponseWriter, r *http.Request, params dat
func (api *API) createUser(ctx context.Context, req codersdk.CreateUserRequest) (database.User, uuid.UUID, error) {
var user database.User
return user, req.OrganizationID, api.Database.InTx(func(db database.Store) error {
var orgRoles []string
orgRoles := make([]string, 0)
// If no organization is provided, create a new one for the user.
if req.OrganizationID == uuid.Nil {
organization, err := db.InsertOrganization(ctx, database.InsertOrganizationParams{
Expand Down
0