add discussion URL to group page

Resolves: #867 Related: fedora-infra/freeipa-fas#144 Signed-off-by: Ryan Lerch <[email protected]>
fedora-infra · Feb 27, 2024 · 36bbc09 · 36bbc09
1 parent 7c5dd6b
commit 36bbc09
Show file tree

Hide file tree

Showing 6 changed files with 364 additions and 352 deletions.
diff --git a/news/867.feature b/news/867.feature
@@ -0,0 +1 @@
+Show the Discussion URL in the group infomation page
diff --git a/noggin/representation/group.py b/noggin/representation/group.py
@@ -10,6 +10,7 @@ class Group(Representation):
         "urls": "fasurl",
         "irc_channel": "fasircchannel",
         "mailing_list": "fasmailinglist",
+        "discussion_url": "fasdiscussionurl",
     }
     attr_types = {
         "members": "list",

diff --git a/noggin/templates/group.html b/noggin/templates/group.html
@@ -21,6 +21,9 @@ <h3 class="mb-0">
           {% if group.mailing_list %}
             <div id="group-mailinglist"><i class="fa fa-fw fa-envelope" aria-hidden="true"></i> <a href="mailto:{{group.mailing_list}}">{{group.mailing_list}}</a></div>
           {% endif %}
+          {% if group.discussion_url %}
+          <div id="group-discussion_url"><i class="fa fa-fw fa-comment-o" aria-hidden="true"></i> <a href="{{group.discussion_url}}">{{group.discussion_url}}</a></div>
+          {% endif %}
           {% if group.irc_channel %}
             <div id="group-ircchannel"><i class="fa fa-fw fa-comments-o" aria-hidden="true"></i> {{ group.irc_channel | channel }}</div>
           {% endif %}

diff --git a/tests/unit/conftest.py b/tests/unit/conftest.py
@@ -211,6 +211,7 @@ def _make_group(name):
             fasurl=f"http://{name}.unit.tests",
             fasmailinglist=f"{name}@lists.unit.tests",
             fasircchannel=f"irc://irc.unit.tests/#{name}",
+            fasdiscussionurl=f"http://discussion.{name}.unit.tests",
         )
         created.append(name)
         return result