paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-22 07:21:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

integrations: Refactor Github Integration to use get_sender_name.

Browse Source
This commit is contained in:
dhruv302003
2023-06-03 18:02:42 +05:30
committed by Tim Abbott
parent 67125acaa6
commit 34e16c1553
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
zerver/webhooks/github/view.py
View File

@@ -364,7 +364,7 @@ def get_team_body(helper: Helper) -> str:
payload = helper.payload payload = helper.payload
changes = payload["changes"] changes = payload["changes"]
if "description" in changes: if "description" in changes:
actor = payload["sender"]["login"].tame(check_string) actor = get_sender_name(payload)
new_description = payload["team"]["description"].tame(check_string) new_description = payload["team"]["description"].tame(check_string)
return f"**{actor}** changed the team description to:\n```quote\n{new_description}\n```" return f"**{actor}** changed the team description to:\n```quote\n{new_description}\n```"
if "name" in changes: if "name" in changes:
@@ -599,7 +599,7 @@ def get_star_body(helper: Helper) -> str:
payload = helper.payload payload = helper.payload
template = "[{user}]({user_url}) {action} the repository [{repo}]({url})." template = "[{user}]({user_url}) {action} the repository [{repo}]({url})."
return template.format( return template.format(
user=payload["sender"]["login"].tame(check_string), user=get_sender_name(payload),
user_url=get_sender_url(payload), user_url=get_sender_url(payload),
action="starred" if payload["action"].tame(check_string) == "created" else "unstarred", action="starred" if payload["action"].tame(check_string) == "created" else "unstarred",
repo=get_repository_full_name(payload), repo=get_repository_full_name(payload),