Commit 228feaaf53a1aa0d1ae5c912c147d70b012d862d
Committed by
Sergio Oliveira
1 parent
4c3b6536
Exists in
master
and in
39 other branches
Added workaround for adding group members
Showing
1 changed file
with
5 additions
and
2 deletions
Show diff stats
src/proxy/templates/proxy/gitlab.html
@@ -26,14 +26,17 @@ | @@ -26,14 +26,17 @@ | ||
26 | {% block head_js %} | 26 | {% block head_js %} |
27 | <script type="text/javascript"> | 27 | <script type="text/javascript"> |
28 | $(function(){ | 28 | $(function(){ |
29 | - // Somehow when there's a merge request, all options just disapear | 29 | + // TODO: Somehow when there's a merge request, all options just disapear |
30 | // so this fix makes it show up again | 30 | // so this fix makes it show up again |
31 | $('div.panel-body div.automerge_widget').removeClass('hide'); | 31 | $('div.panel-body div.automerge_widget').removeClass('hide'); |
32 | 32 | ||
33 | - // Somehow the dangerous settings are not showing up for default | 33 | + // TODO: Somehow the dangerous settings are not showing up for default |
34 | // but for a mistery, they come back to live if the first status is | 34 | // but for a mistery, they come back to live if the first status is |
35 | // not-hide | 35 | // not-hide |
36 | $('div.danger-settings div.js-toggle-content').removeClass('hide'); | 36 | $('div.danger-settings div.js-toggle-content').removeClass('hide'); |
37 | + | ||
38 | + // TODO: For the same reason from the two walkarounds above | ||
39 | + $('div.new-group-member-holder').removeClass('hide'); | ||
37 | }); | 40 | }); |
38 | </script> | 41 | </script> |
39 | <script type="text/javascript" src="{% static 'third-party/bootstrap/js/bootstrap.min.js' %}"></script> | 42 | <script type="text/javascript" src="{% static 'third-party/bootstrap/js/bootstrap.min.js' %}"></script> |