summaryrefslogtreecommitdiff
path: root/app/controllers
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' into mr-merge-basemr-merge-baseDouwe Maan2016-01-285-11/+10
|\
| * Merge branch 'events-performance' into 'master' Robert Speicher2016-01-273-9/+5
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve performance of retrieving last update times for events See 75195eefcca4cb3961986c90c8e837da02f30134 and https://gitlab.com/gitlab-org/gitlab-ce/issues/12415#note_3387317 for the juicy details. Related issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/12415 See merge request !2613
| | * Don't pluck project IDs for eventsYorick Peterse2016-01-263-9/+5
| | | | | | | | | | | | | | | By instead using a sub-query we save ourselves the overhead of loading any data into memory only to pass it on to another query.
| * | Merge branch 'rs-skip-2fa-on-signout' into 'master' Robert Speicher2016-01-271-0/+2
| |\ \ | | | | | | | | | | | | | | | | Skip the 2FA requirement during logout See merge request !2581
| | * | Skip the 2FA requirement during logoutrs-skip-2fa-on-signoutRobert Speicher2016-01-231-0/+2
| | | |
| * | | Merge branch 'fix/unexistent-group-500' into 'master' Robert Speicher2016-01-271-0/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Return a 404 instead of a 500 when accessing a non-existent group Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/3352 Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/5600 See merge request !2614
| | * \ \ Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-01-271-4/+5
| | |\ \ \ | | | | | | | | | | | | | | | | | | fix/unexistent-group-500
| | * | | | update fix based on feedbackJames Lopez2016-01-271-2/+1
| | | | | |
| | * | | | fix for failing specJames Lopez2016-01-261-1/+2
| | | | | |
| | * | | | feature and fix for 500 on group missingJames Lopez2016-01-261-0/+1
| | | |_|/ | | |/| |
| * | | | Merge branch 'rs-2fa-requirement-wording' into 'master' Dmitriy Zaporozhets2016-01-271-2/+2
| |\ \ \ \ | | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | Update wording for 2FA requirement notice See merge request !2582
| | * | | Update wording for 2FA requirement noticers-2fa-requirement-wordingRobert Speicher2016-01-231-2/+2
| | | |/ | | |/|
* | | | Correctly determine MR diff base when MR has merge conflictsDouwe Maan2016-01-271-1/+1
|/ / /
* | | Merge branch 'rs-groups-index' into 'master' Douwe Maan2016-01-261-4/+5
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the `/groups` route behave as expected The route is supposed to redirect the Groups#index request based on whether or not a user was logged in. If they are, we redirect them to their groups dashboard; if they're not, we redirect them to the public Explore page. But due to overly aggressive `before_action`s that weren't excluding (or including) the `index` action, the request always resulted in a 404, whether a user was logged in or not. Closes #12660 See merge request !2580
| * | Make the `/groups` route behave as expectedrs-groups-indexRobert Speicher2016-01-231-4/+5
| |/ | | | | | | | | | | | | | | | | | | | | | | | | The route is supposed to redirect the Groups#index request based on whether or not a user was logged in. If they are, we redirect them to their groups dashboard; if they're not, we redirect them to the public Explore page. But due to overly aggressive `before_action`s that weren't excluding the `index` action, the request always resulted in a 404, whether a user was logged in or not. Closes #12660
* | Fixi import redirect loopDouglas Barbosa Alexandre2016-01-251-5/+15
|/
* Merge branch 'fix-comment-on-diff-ajax-loading'Dmitriy Zaporozhets2016-01-221-12/+10
|\
| * Merge remote-tracking branch 'origin/master' into ↵Rémy Coutable2016-01-203-0/+20
| |\ | | | | | | | | | fix-comment-on-diff-ajax-loading
| * | Fix diff comments loaded by AJAX to load comment with diff in discussion tabRémy Coutable2016-01-191-12/+10
| | | | | | | | | | | | | | | | | | | | | This commits also fixes two minor issues: - Ensure notes that the current user is not allowed to see are not returned in the AJAX notes loading - Ensure the notes counter badge is decremented of 1 instead of 2
* | | Merge branch 'fix-mr-diff-edit-button' into 'master' Robert Speicher2016-01-221-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix MR diff 'Edit' button It now works and has the right size. See merge request !2542
| * | | Fix MR diff 'Edit' buttonfix-mr-diff-edit-buttonDouwe Maan2016-01-211-1/+1
| | | |
* | | | Make sure non-highlighted diffs are still escapedunescaped-diffsDouwe Maan2016-01-221-1/+3
|/ / /
* | | Merge branch 'master' into issue_3945issue_3945Rubén Dávila2016-01-202-0/+7
|\ \ \ | | |/ | |/|
| * | Merge branch 'feature/check-against-rbl-only' into 'master'Robert Speicher2016-01-202-0/+7
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Split from !2455 References #9092 See merge request !2515
| | * | Add IP blocking against DNSBL at sign-upfeature/check-against-rbl-onlyTomasz Maczukin2016-01-202-0/+7
| | |/
* | | Merge branch 'master' into issue_3945Douwe Maan2016-01-202-0/+13
|\ \ \ | |/ /
| * | Send user context to SentryDouwe Maan2016-01-201-0/+11
| | |
| * | Add sentry integrationJeroen Nijhof2016-01-181-0/+2
| |/
* | Save and use actual diff base commit for MR diff highlightingDouwe Maan2016-01-202-4/+8
| |
* | Merge branch 'master' into issue_3945Douwe Maan2016-01-208-7/+9
|\ \ | |/
| * Autofill abuse message text with user url. Closes #2838Josh Frye2016-01-161-0/+1
| |
| * Merge branch 'fix-autocomplete-new-issues' into 'master' Robert Speicher2016-01-153-2/+3
| |\ | | | | | | | | | | | | | | | | | | Fix autocomplete for new issues/MRs/snippets Fixes gitlab-org/gitlab-ce#8067 See merge request !2446
| | * Fix autocomplete for new issues/MRs/snippetsfix-autocomplete-new-issuesYorick Peterse2016-01-153-2/+3
| | |
| * | Merge branch 'feature/allow-artifacts-for-reporters' into 'master' Douwe Maan2016-01-153-4/+4
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow access to artifacts for users with reporter role This is originally introduced by @ajohnsn in this merge request: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2259 I've added and refactored specs, original commit by @ajohnsn has been cherry picked here. See merge request !2448
| | * | Fix nonexistent method in artifacts controllerGrzegorz Bizon2016-01-151-1/+1
| | | |
| | * | Give reporters the ability to download artifacts.Andrew Johnson2016-01-152-3/+3
| | |/ | | | | | | | | | Also fix a few places where page_404 should be render_404.
| * | Merge branch 'issue_7975' into 'master' Drew Blessing2016-01-151-1/+1
| |\ \ | | |/ | |/| | | | | | | | | | | | | Consider that URL can end with '/' before redirecting. Closes #7975 See merge request !2416
| | * Consider that URL can end with '/' before redirecting. #7975issue_7975Rubén Dávila2016-01-131-1/+1
| | |
* | | Merge branch 'master' into issue_3945Rubén Dávila2016-01-1423-66/+195
|\ \ \ | |/ /
| * | Merge branch 'unsubscribe-from-thread-from-email-footer' into 'master' Douwe Maan2016-01-141-0/+25
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unsubscribe from thread through link in email footer Closes #3437 ![Screenshot_from_2015-12-11_15-35-06](/uploads/cfb3d8737d4757f527995411f103d0ce/Screenshot_from_2015-12-11_15-35-06.png) ![Screenshot_from_2015-12-11_15-35-56](/uploads/9b7121be7ce4b05e5995ca6d38c5bea4/Screenshot_from_2015-12-11_15-35-56.png) See merge request !2068
| | * | #can_unsubscribe? -> #?unsubscribable?Zeger-Jan van de Weg2016-01-111-1/+1
| | | |
| | * | Minor improvements, unsubscribe from email footerZeger-Jan van de Weg2016-01-091-2/+2
| | | |
| | * | Unsubscribe from thread through link in email footerZeger-Jan van de Weg2016-01-091-0/+25
| | | |
| * | | Improve readability of artifacts browser `Entry` related codeGrzegorz Bizon2016-01-141-5/+5
| | | |
| * | | Improvements, readability for artifacts browserGrzegorz Bizon2016-01-141-7/+7
| | | |
| * | | Check if file exists in metadata in download actionGrzegorz Bizon2016-01-141-3/+8
| | | |
| * | | Render only valid paths in artifacts metadataGrzegorz Bizon2016-01-141-1/+4
| | | | | | | | | | | | | | | | | | | | In this version we will support only relative paths in artifacts metadata. Support for absolute paths will be introduced later.
| * | | Simplify implementation of build artifacts browser (refactoring)Grzegorz Bizon2016-01-141-1/+7
| | | |
| * | | Add method that checks if path exists in `StringPath`Grzegorz Bizon2016-01-141-1/+1
| | | |
| * | | Render 404 when artifacts path is invalidGrzegorz Bizon2016-01-141-0/+1
| | | |