Skip to content

Commit 7feee00

Browse files
committed
discord: when announcing in #releases, also provide a link to the forum topic
1 parent 11ede43 commit 7feee00

File tree

3 files changed

+29
-13
lines changed

3 files changed

+29
-13
lines changed

appstore/developer_portal_api.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -206,18 +206,18 @@ def submit_new_app():
206206
if algolia_index and app_obj.visible:
207207
algolia_index.partial_update_objects([algolia_app(app_obj)], { 'createIfNotExists': True })
208208

209-
try:
210-
discord.announce_new_app(app_obj, pbw.is_generated())
211-
except Exception as e:
212-
# We don't want to fail just because Discord is being weird
213-
print("Discord is being weird: {repr(e)}")
214-
215209
try:
216210
discourse.announce_new_app(app_obj, pbw.is_generated())
217211
except Exception as e:
218212
# We don't want to fail just because Discourse is being weird
219213
print("Discourse is being weird: {repr(e)}")
220214

215+
try:
216+
discord.announce_new_app(app_obj, pbw.is_generated())
217+
except Exception as e:
218+
# We don't want to fail just because Discord is being weird
219+
print("Discord is being weird: {repr(e)}")
220+
221221
return jsonify(success=True, id=app_obj.id)
222222

223223

appstore/discord.py

Lines changed: 22 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
from .settings import config
66
from .utils import get_app_description, generate_image_url, who_am_i
7+
from .discourse import get_topic_url_for_app
78

89
party_time_emoji = ["🎉","🥳","👏","❤️","🥰","🎊"]
910

@@ -18,6 +19,19 @@ def announce_release(app, release, is_generated):
1819
release_notes = release.release_notes
1920
if not release_notes:
2021
release_notes = "N/A"
22+
23+
request_fields = [{
24+
"name": "Release Notes",
25+
"value": release_notes
26+
}]
27+
28+
topic_url = get_topic_url_for_app(app)
29+
if topic_url:
30+
request_fields.append({
31+
"name": "Discuss it on the Rebble Dev Forum!",
32+
"value": topic_url
33+
})
34+
2135

2236
request_data = {
2337
"embeds": [{
@@ -29,12 +43,7 @@ def announce_release(app, release, is_generated):
2943
"width": 80
3044
},
3145
"description": f"{app.developer.name} just updated their {app.type} *{app.title}* to version {release.version}!",
32-
"fields": [
33-
{
34-
"name": "Release Notes",
35-
"value": release_notes
36-
}
37-
]
46+
"fields": request_fields
3847
}]
3948
}
4049

@@ -76,6 +85,13 @@ def announce_new_app(app, is_generated):
7685
"name": "Website",
7786
"value": app.website
7887
})
88+
89+
topic_url = get_topic_url_for_app(app)
90+
if topic_url:
91+
request_fields.append({
92+
"name": "Discuss it on the Rebble Dev Forum!",
93+
"value": topic_url
94+
})
7995

8096
txt_type = app.type if not is_generated else "Generated Watchface"
8197

appstore/discourse.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,4 +164,4 @@ def user_owns_discourse_topic(discourse_topic_url):
164164
me = auth_result.json()
165165
my_username = me["rebble_username"]
166166

167-
return discourse_username == my_username
167+
return discourse_username == my_username

0 commit comments

Comments
 (0)