diff options
author | Magnus Hagander | 2019-02-06 20:36:01 +0000 |
---|---|---|
committer | Magnus Hagander | 2019-02-06 20:36:01 +0000 |
commit | e937795ea90ba1a1b72fed942ef385b285c3a378 (patch) | |
tree | fbe56f1fd9394db4f4d352562a9ad9b4ff6fc81c /pgcommitfest/commitfest/feeds.py | |
parent | a3bac5922db76efd5b6bb331a7141e9ca3209c4a (diff) |
Fix activity feed queries
These clearly had little to do with reality since they would return
duplicate entries if a patch was in more than one cf..
Diffstat (limited to 'pgcommitfest/commitfest/feeds.py')
-rw-r--r-- | pgcommitfest/commitfest/feeds.py | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/pgcommitfest/commitfest/feeds.py b/pgcommitfest/commitfest/feeds.py index 969de69..aa950fb 100644 --- a/pgcommitfest/commitfest/feeds.py +++ b/pgcommitfest/commitfest/feeds.py @@ -18,20 +18,14 @@ class ActivityFeed(Feed): return self.activity def item_title(self, item): - if self.cfid: - return item['name'] - else: - return '{cfname}: {name}'.format(**item) + return item['name'] def item_description(self, item): - if self.cfid: - return "<div>Patch: {name}</div><div>User: {by}</div>\n<div>{what}</div>".format(**item) - else: - return "<div>Commitfest: {cfname}</div><div>Patch: {name}</div><div>User: {by}</div><div>{what}</div>".format(**item) + return "<div>Patch: {name}</div><div>User: {by}</div>\n<div>{what}</div>".format(**item) def item_link(self, item): if self.cfid: - return 'https://commitfest.postgresql.org/{cfid}/{patchid}/'.format(cfid=self.cfid, **item) + return 'https://commitfest.postgresql.org/{0}/{1}/'.format(self.cfid, item['patchid']) else: return 'https://commitfest.postgresql.org/{cfid}/{patchid}/'.format(**item) |