Commit Graph

8 Commits

Author SHA1 Message Date
Fatih Acet 279c61f3cb Merge branch 'cs-remove-inline-js' into 'master'
Remove two simple instances of inline JavaScript.

Not very complicated, they're moved to the dispatcher.

See merge request !5513
2016-08-10 22:11:06 +00:00
Alfredo Sumaran 8f29c332b5 Wrap classes with IIFE and define it inside gl namespace 2016-08-05 17:19:01 -05:00
Alfredo Sumaran 2b2c42a1fa Refactor of Protected Branch Edit List 2016-08-05 17:19:01 -05:00
Alfredo Sumaran 1ac953dab4 Update layout and JS for create protected branch.
Also updates protect branch list
2016-08-05 17:19:01 -05:00
Timothy Andrew cebcc417ed Implement final review comments from @rymai.
1. Instantiate `ProtectedBranchesAccessSelect` from `dispatcher`

2. Use `can?(user, ...)` instead of `user.can?(...)`

3. Add `DOWNTIME` notes to all migrations added in !5081.

4. Add an explicit `down` method for migrations removing the
   `developers_can_push` and `developers_can_merge` columns, ensuring that
   the columns created (on rollback) have the appropriate defaults.

5. Remove duplicate CHANGELOG entries.

6. Blank lines after guard clauses.
2016-07-29 15:20:39 +05:30
Alfredo Sumaran 8f3e3f6b80 Make sure Labels is instantiated on edit page 2016-07-27 03:24:11 -05:00
Connor Shea df0a77a7be
Removes two simple instances of inline JavaScript. 2016-07-26 16:59:55 -06:00
Fatih Acet aaa9509d12 ES6ify all the things! 2016-07-24 23:45:11 +03:00