Compare commits
No commits in common. "fee3a6ccb03601ac49c4aae6e55070fd7189970d" and "a40ce94d8c2dcd33544f91b945add9315a507c72" have entirely different histories.
fee3a6ccb0
...
a40ce94d8c
1 changed files with 2 additions and 12 deletions
|
|
@ -72,22 +72,12 @@ 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 as exc:
|
except Exception:
|
||||||
raise MattermostError(
|
pass # already a member or team not configured — not fatal
|
||||||
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:
|
||||||
"""
|
"""
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue