Skip to content

feat: notify about created user account #14010

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 26 commits into from
Jul 30, 2024
Merged

feat: notify about created user account #14010

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
c18ada0
feat: notify about created user account
mtojek Jul 25, 2024
1cc3adb
Merge branch 'main' into 17-account-notifs
mtojek Jul 26, 2024
ca2bdde
migrations
mtojek Jul 26, 2024
d6e8964
Username
mtojek Jul 26, 2024
2d74c4e
events
mtojek Jul 26, 2024
3c3b5af
fix
mtojek Jul 26, 2024
312d9fa
WIP
mtojek Jul 26, 2024
82ec37f
fix version
mtojek Jul 26, 2024
d7b2c73
simplify
mtojek Jul 29, 2024
b81bb6a
WIP
mtojek Jul 29, 2024
00275dd
fix: versions
mtojek Jul 29, 2024
e356ba8
test
mtojek Jul 29, 2024
6bc1d2d
fix test
mtojek Jul 29, 2024
67a5137
users notified
mtojek Jul 29, 2024
6431a0e
Merge branch 'main' into 17-account-notifs
mtojek Jul 29, 2024
42d9ba1
post merge
mtojek Jul 29, 2024
ecc7d30
fmt
mtojek Jul 29, 2024
b2dcb3b
skip notif
mtojek Jul 29, 2024
70e2d2c
Danny's feedback
mtojek Jul 30, 2024
8696b70
given when then
mtojek Jul 30, 2024
a92d059
Fix tests
mtojek Jul 30, 2024
b5b0d90
Skip notifications
mtojek Jul 30, 2024
2de1786
Merge branch 'main' into 17-account-notifs
mtojek Jul 30, 2024
b843232
fix: created_account_name
mtojek Jul 30, 2024
2a53a0b
api.Database -> tx
mtojek Jul 30, 2024
77c0c28
pq issue
mtojek Jul 30, 2024
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
pq issue
  • Loading branch information
@mtojek
mtojek committed Jul 30, 2024
commit 77c0c28e8af6e863a892f6045ccaca26d44e4180
36 changes: 11 additions & 25 deletions coderd/users.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import (
"github.com/go-chi/chi/v5"
"github.com/go-chi/render"
"github.com/google/uuid"
"golang.org/x/sync/errgroup"
"golang.org/x/xerrors"

"cdr.dev/slog"
Expand Down Expand Up @@ -1281,33 +1280,20 @@ func (api *API) CreateUser(ctx context.Context, store database.Store, req Create
return user, req.OrganizationID, err
}

// Notify user admins
// Get all users with user admin permission including owners
var owners, userAdmins []database.GetUsersRow
var eg errgroup.Group
eg.Go(func() error {
var err error
owners, err = store.GetUsers(ctx, database.GetUsersParams{
RbacRole: []string{codersdk.RoleOwner},
})
if err != nil {
return xerrors.Errorf("get owners: %w", err)
}
return nil
// Notify all users with user admin permission including owners
// Notice: we can't scrape the user information in parallel as pq
// fails with: unexpected describe rows response: 'D'
owners, err := store.GetUsers(ctx, database.GetUsersParams{
RbacRole: []string{codersdk.RoleOwner},
})
eg.Go(func() error {
var err error
userAdmins, err = store.GetUsers(ctx, database.GetUsersParams{
RbacRole: []string{codersdk.RoleUserAdmin},
})
if err != nil {
return xerrors.Errorf("get user admins: %w", err)
}
return nil
if err != nil {
return user, req.OrganizationID, xerrors.Errorf("get owners: %w", err)
}
userAdmins, err := store.GetUsers(ctx, database.GetUsersParams{
RbacRole: []string{codersdk.RoleUserAdmin},
})
err = eg.Wait()
if err != nil {
return database.User{}, uuid.Nil, err
return user, req.OrganizationID, xerrors.Errorf("get user admins: %w", err)
}

for _, u := range append(owners, userAdmins...) {
Expand Down
Loading