jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/348386 )

Change subject: Add links to user's accounts to membership request
......................................................................


Add links to user's accounts to membership request

Make checking up on the validity of a membership request a bit easier by
providing links to the user's wikitech user page, and if known their SUL
account status and phabricator profile.

Bug: T162508
Change-Id: Ic052c413c57306d96c249e6edd80400e66fd2086
---
M striker/templates/tools/membership/status.html
M striker/tools/views.py
2 files changed, 12 insertions(+), 1 deletion(-)

Approvals:
  BryanDavis: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/striker/templates/tools/membership/status.html 
b/striker/templates/tools/membership/status.html
index f697fd7..2bc658b 100644
--- a/striker/templates/tools/membership/status.html
+++ b/striker/templates/tools/membership/status.html
@@ -19,7 +19,16 @@
     <div class="panel-body">
       <dl class="dl-horizontal">
         <dt>{% trans "User" %}</dt>
-        <dd>{{ app.user }}</dd>
+        <dd>{{ app.user }} ({{ app.user.shellname }})</dd>
+        <dd>
+          <a href="{{ wikitech }}/wiki/User:{{ app.user.ldapname|urlencode 
}}">{% trans "wikitech" %}</a>
+          {% if app.user.sulname %}
+          | <a href="{{ meta }}?title={{ 'Special:CentralAuth'|urlencode 
}}&target={{ app.user.sulname|urlencode }}">{% trans "SUL" %}</a>
+          {% endif %}
+          {% if app.user.phaburl %}
+          | <a href="{{ app.user.phaburl }}">{% trans "phab" %}</a>
+          {% endif %}
+        </dd>
         <dt>{% trans "Status" %}</dt>
         <dd>{{ app.get_status_display }}</dd>
         <dt>{% trans "Reason" %}</dt>
diff --git a/striker/tools/views.py b/striker/tools/views.py
index d0afa28..1541beb 100644
--- a/striker/tools/views.py
+++ b/striker/tools/views.py
@@ -340,5 +340,7 @@
     ctx = {
         'app': request,
         'form': form,
+        'wikitech': settings.WIKITECH_URL,
+        'meta': settings.OAUTH_MWURL,
     }
     return shortcuts.render(req, 'tools/membership/status.html', ctx)

-- 
To view, visit https://gerrit.wikimedia.org/r/348386
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic052c413c57306d96c249e6edd80400e66fd2086
Gerrit-PatchSet: 2
Gerrit-Project: labs/striker
Gerrit-Branch: master
Gerrit-Owner: BryanDavis <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: Madhuvishy <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Tim Landscheidt <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to