Merge pull request 'V0.7 de l'amélioration du suivi des issues' (#258) from dev into main
Reviewed-on: #258
This commit is contained in:
commit
9cd4398cbb
1 changed files with 13 additions and 1 deletions
|
|
@ -450,18 +450,25 @@ class BugReport(commands.Cog):
|
||||||
notified_count = 0
|
notified_count = 0
|
||||||
for issue_iid, issue_data in list(self.issue_data.items()):
|
for issue_iid, issue_data in list(self.issue_data.items()):
|
||||||
if not issue_data.get("pending_deployment_notification"):
|
if not issue_data.get("pending_deployment_notification"):
|
||||||
|
kuby_logger.debug(f"No pending deployment notification for issue #{issue_iid}")
|
||||||
continue
|
continue
|
||||||
|
|
||||||
requester_id = issue_data.get("requester_id")
|
requester_id = issue_data.get("requester_id")
|
||||||
if not requester_id:
|
if not requester_id:
|
||||||
|
kuby_logger.warning(f"Missing requester_id for pending deployment notification on issue #{issue_iid}")
|
||||||
continue
|
continue
|
||||||
|
|
||||||
|
kuby_logger.info(f"Processing deployment notification for issue #{issue_iid} to requester_id={requester_id}")
|
||||||
user = self.bot.get_user(requester_id)
|
user = self.bot.get_user(requester_id)
|
||||||
if not user:
|
if not user:
|
||||||
try:
|
try:
|
||||||
user = await self.bot.fetch_user(requester_id)
|
user = await self.bot.fetch_user(requester_id)
|
||||||
except Exception:
|
kuby_logger.info(f"Fetched user {user} for issue #{issue_iid}")
|
||||||
|
except Exception as fetch_error:
|
||||||
|
kuby_logger.warning(f"Unable to fetch user {requester_id} for issue #{issue_iid}: {fetch_error}")
|
||||||
continue
|
continue
|
||||||
|
else:
|
||||||
|
kuby_logger.info(f"Resolved cached user {user} for issue #{issue_iid}")
|
||||||
|
|
||||||
try:
|
try:
|
||||||
embed = disnake.Embed(
|
embed = disnake.Embed(
|
||||||
|
|
@ -476,10 +483,15 @@ class BugReport(commands.Cog):
|
||||||
if avatar:
|
if avatar:
|
||||||
embed.set_thumbnail(url=avatar.url)
|
embed.set_thumbnail(url=avatar.url)
|
||||||
|
|
||||||
|
kuby_logger.info(f"Attempting to DM deployment notification to {user} for issue #{issue_iid}")
|
||||||
await user.send(embed=embed)
|
await user.send(embed=embed)
|
||||||
issue_data["pending_deployment_notification"] = False
|
issue_data["pending_deployment_notification"] = False
|
||||||
notified_count += 1
|
notified_count += 1
|
||||||
kuby_logger.info(f"Notification de déploiement envoyée à {user} pour l'issue #{issue_iid}")
|
kuby_logger.info(f"Notification de déploiement envoyée à {user} pour l'issue #{issue_iid}")
|
||||||
|
except disnake.Forbidden as forbidden_error:
|
||||||
|
kuby_logger.warning(f"DM forbidden for {user} on issue #{issue_iid}: {forbidden_error}")
|
||||||
|
except disnake.HTTPException as http_error:
|
||||||
|
kuby_logger.warning(f"Discord HTTP error while sending deployment notification to {user} on issue #{issue_iid}: {http_error}")
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
kuby_logger.warning(f"Impossible d'envoyer la notification de déploiement à {user} pour l'issue #{issue_iid}: {e}")
|
kuby_logger.warning(f"Impossible d'envoyer la notification de déploiement à {user} pour l'issue #{issue_iid}: {e}")
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue