fix(sync-gitlab): Pass Team in call to remove_team_member instead of the team ID

irgeek · irgeek · commit cc172cc2d242 · 2025-06-30T16:46:46.000+02:00
diff --git a/team-mapping-gitlab-gitguardian/sync_gitlab.py b/team-mapping-gitlab-gitguardian/sync_gitlab.py
@@ -325,7 +325,6 @@ def synchronize_team_members(
                     invitation_by_id[team_invitation.invitation_id].email,
                 )
 
-            team_id = gg_team.id
             for member in members_to_add:
                 # If the member exists in GitGuardian, we can add him to the team
                 if member in members_by_emails:
@@ -351,7 +350,7 @@ def synchronize_team_members(
                     None,
                 )
                 if team_member_id:
-                    remove_team_member(team_id, team_member_id)
+                    remove_team_member(gg_team, team_member_id)
 
 
 def infer_gitlab_email(

Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,6 @@ def synchronize_team_members(`
`325`	`325`	`invitation_by_id[team_invitation.invitation_id].email,`
`326`	`326`	`)`
`327`	`327`
`328`		`- team_id = gg_team.id`
`329`	`328`	`for member in members_to_add:`
`330`	`329`	`# If the member exists in GitGuardian, we can add him to the team`
`331`	`330`	`if member in members_by_emails:`
`@@ -351,7 +350,7 @@ def synchronize_team_members(`
`351`	`350`	`None,`
`352`	`351`	`)`
`353`	`352`	`if team_member_id:`
`354`		`- remove_team_member(team_id, team_member_id)`
	`353`	`+ remove_team_member(gg_team, team_member_id)`
`355`	`354`
`356`	`355`
`357`	`356`	`def infer_gitlab_email(`