vikunja/vikunja
6
43
Fork 73
Code Issues 150 Pull Requests 20 Packages Releases 32 Activity

feat: assign users to teams via OIDC claims #1393

Merged
konrad merged 93 commits from viehlieb/api:950_reworked_assign_teams_via_oidc into main 2024-03-02 08:47:12 +00:00
Conversation 34 Commits 93 Files Changed 26 +2 -7
1 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 765cfae63a - Show all commits

9
pkg/models/team_members.go
View File

@ -110,18 +110,13 @@ func (tm *TeamMember) Delete(s *xorm.Session, _ web.Auth) (err error) {
}
func (tm *TeamMember) CheckMembership(s *xorm.Session) (exists bool, err error) {
member, err := user2.GetUserByUsername(s, tm.Username)
if err != nil {
return
}
tm.UserID = member.ID
exists, err = s.
viehlieb marked this conversation as resolved Outdated
konrad commented 2023-02-13 10:18:22 +00:00
Outdated
Review
Copy Link

Can you do this either with a join to avoid loading the user or at least use the "simple" function?

Can you do this either with a join to avoid loading the user or at least use the "simple" function?
viehlieb commented 2023-02-13 17:49:47 +00:00
Outdated
Review
Copy Link

done.

done.
Where("team_id = ? AND user_id = ?", tm.TeamID, tm.UserID).
viehlieb marked this conversation as resolved Outdated
konrad commented 2023-02-14 11:37:43 +00:00
Outdated
Review
Copy Link

This should be Exist(&TeamMember{}).

This should be `Exist(&TeamMember{})`.
viehlieb commented 2023-02-23 14:48:53 +00:00
Outdated
Review
Copy Link

done

done
Get(&TeamMember{})
if exists {
if exists && err == nil {
return
}
return exists, ErrUserIsMemberOfTeam{tm.UserID, tm.UserID}
return exists, err
}
viehlieb marked this conversation as resolved Outdated
konrad commented 2023-02-23 15:24:19 +00:00
Outdated
Review
Copy Link

Couldn't you just return the result here directly?

Couldn't you just return the result here directly?
viehlieb commented 2023-03-10 12:10:00 +00:00
Outdated
Review
Copy Link

In fact this function is not needed anymore for this feature, so I'll just remove it.
It was used for finding out whether a user should be signed out from team or deleted.

In fact this function is not needed anymore for this feature, so I'll just remove it. It was used for finding out whether a user should be signed out from team or deleted.
👍 1
// Update toggles a team member's admin status