addwin/chat
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request 'fix: surface real errors when adding user to Mattermost team' (#11) from feature/debug-team-member into master

Browse source 
Reviewed-on: #11
This commit is contained in:
Ghasemi 2026-08-09 06:41:41 -04:00
commit fee3a6ccb0
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
apps/chat/integrations/mattermost/client.py
View file

@ -72,12 +72,22 @@ class MattermostClient:
def _ensure_team_member(self, mm_user_id: str) -> None: def _ensure_team_member(self, mm_user_id: str) -> None:
if not self._team_id: if not self._team_id:
return return
try:
self._driver.teams.get_team_member(self._team_id, mm_user_id)
return # already a member
except ResourceNotFound:
pass
except Exception as exc:
raise MattermostError(f"Team membership lookup failed: {exc}") from exc
try: try:
self._driver.teams.add_user_to_team( self._driver.teams.add_user_to_team(
self._team_id, options={"team_id": self._team_id, "user_id": mm_user_id} self._team_id, options={"team_id": self._team_id, "user_id": mm_user_id}
) )
except Exception: except Exception as exc:
pass # already a member or team not configured — not fatal raise MattermostError(
f"Adding user {mm_user_id} to team failed: {exc}"
) from exc
def get_or_create_user(self, user_id: UUID) -> str: def get_or_create_user(self, user_id: UUID) -> str:
""" """