Commit 0fbfa5bbf4a9aa03f31e0391f7b24d34d6e2801e
Exists in
master
and in
39 other branches
Merge pull request #40 from colab-community/gitlab-fixes
Added workaround for adding group members
Showing
1 changed file
with
5 additions
and
2 deletions
Show diff stats
src/proxy/gitlab/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> |