summaryrefslogtreecommitdiff
path: root/features
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'issues-show-performance' into 'master' Robert Speicher2016-03-212-1/+3
|\ | | | | | | | | | | | | | | | | | | | | | | Improve performance of viewing individual issues This MR does two things: 1. `Issue#related_branches` no longer performs Git operations that aren't needed 2. The output of `Repository#exists?` is now cached and flushed properly Combined these two changes should further cut down the amount of Git operations performed when viewing individual issues (and possibly other pages). See merge request !3296
| * Create SSH keys for SSH clone Spinach testsissues-show-performanceYorick Peterse2016-03-212-0/+2
| | | | | | | | | | | | | | | | | | | | These tests would check if the "This project is empty" banner would contain SSH clone URLs. Oddly enough this should have never passed (as far as I can tell) as SSH clone URLs in this banner are _only_ displayed if the current user has at least 1 SSH key attached. Since the tests never seem to create any they never should have passed, yet somehow they did. To solve this the Spinach tests in question now ensure at least 1 SSH key is present.
| * Create repositories in Spinach issues testsYorick Peterse2016-03-191-1/+1
| | | | | | | | | | | | Similar to ad90dba5185e30883d5ad6008c166b0df0108ebf we always have a repository in the real world, so let's also create one in our Spinach tests.
* | Fix specs and remove unnecessary console.logStan Hu2016-03-201-2/+2
|/
* Merge branch 'fix-mr-close-button' into 'master' Jacob Schatz2016-03-181-0/+8
|\ | | | | | | | | | | | | Fixes issue with close button not working on MR Closes #14383 See merge request !3295
| * Added testPhil Hughes2016-03-181-0/+8
| |
* | Merge branch 'confidential-issues' into 'master' Douwe Maan2016-03-181-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Add confidential issues Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/3678 More information: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/227 See merge request !3282
| * | Restrict access to confidential issues on activity feedDouglas Barbosa Alexandre2016-03-171-1/+1
| | |
* | | Merge branch 'optimistic-todos' into 'master' Jacob Schatz2016-03-181-1/+0
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds small AJAX optimistic functionality to todos. Fixes #13656 <br/> A good first step and boring solution. <br/> Will make ajax call to remove each issue. <br/> If issue is last in group of issues will refresh page. <br/> If issues remain in group will remove row with JS. <br/> Adds loading spinner to button and disables. <br/> ![todos-optimistic](/uploads/c6aec4a688e5125d4df55a2a3a55d4d7/todos-optimistic.gif) <br/> cc @dzaporozhets @JobV @dbalexandre See merge request !2946
| * | Fixed failing testsoptimistic-todosPhil Hughes2016-03-171-1/+0
| | |
* | | Improve award emoji test reliability by checking that the emoji is displayedStan Hu2016-03-172-4/+11
| |/ |/| | | | | before the search field is focused
* | Reuse `no_cache_headers` method in badges controllerGrzegorz Bizon2016-03-171-1/+1
|/
* Fix wiki search results point to raw sourceHiroyuki Sato2016-03-171-1/+1
|
* Merge branch 'fix-capybara-deprecation-warnings' into 'master' Robert Speicher2016-03-161-1/+1
|\ | | | | | | | | Fix Capybara 2.6.2 deprecation warnings See merge request !3236
| * Fix Capybara 2.6.2 deprecation warningsStan Hu2016-03-151-1/+1
| |
* | Improving the original label-subscribing implementationRémy Coutable2016-03-152-11/+10
| | | | | | | | | | | | | | 1. Make the "subscribed" text in Issuable sidebar reflect the labels subscription status 2. Current user mut be logged-in to toggle issue/MR/label subscription
* | Original implementation to allow users to subscribe to labelsTimothy Andrew2016-03-152-0/+50
|/ | | | | | | | | | | | | | | | | | 1. Allow subscribing (the current user) to a label - Refactor the `Subscription` coffeescript class - The main change is that it accepts a container, and conducts all DOM queries within its scope. We need this because the labels page has multiple instances of `Subscription` on the same page. 2. Creating an issue or MR with labels notifies users subscribed to those labels - Label `has_many` subscribers through subscriptions. 3. Adding a label to an issue or MR notifies users subscribed to those labels - This only applies to subscribers of the label that has just been added, not all labels for the issue.
* Merge branch 'revert-avatar-cropping' into 'master' Rémy Coutable2016-03-151-14/+6
|\ | | | | | | | | | | | | Revert the avatar cropping feature Reverts !2951 since the added 'mini_magick' dependency should be replaced by client-side croppping See merge request !3223
| * Revert "Merge branch 'avatar-cropping' into 'master' "revert-avatar-croppingRémy Coutable2016-03-151-14/+6
| | | | | | | | | | This reverts commit 01160fc06182de89c400af174861f6545ad6ceb8, reversing changes made to 4bff9daf8b6d85e9c78565e21cfaa3f6d36f0282.
* | Merge branch 'fix/13928-wrong-iid-of-max-iid' into 'master' Douwe Maan2016-03-153-1/+32
|\ \ | | | | | | | | | | | | | | | | | | | | | Fixes "iid of max iid" in Issuable sidebar for merged MR Fixes #13928. See merge request !3046
| * | Fixes "iid of max iid" in Issuable sidebar for merged MRfix/13928-wrong-iid-of-max-iidRémy Coutable2016-03-083-1/+32
| | | | | | | | | | | | Fixes #13928
* | | Merge branch 'application-settings' into 'master' Robert Speicher2016-03-152-9/+4
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | Applications tab on profile settings Closes #13855 See merge request !3031
| * | Fixed failing application settings testsPhil Hughes2016-03-042-9/+4
| | |
* | | Merge branch 'fix-commit-status-rendering' into 'master' Robert Speicher2016-03-151-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cleanup Ci::Commit, Ci::Build and CommitStatus views This MR tries to do first sweep of cleanups to Ci::Commit and Ci::Build objects removing all view-related functions and fixing the API from other side. Fixes: https://gitlab.com/gitlab-org/gitlab-ce/issues/6046 See merge request !2760
| * | | Cleanup CiCommit and CiBuildKamil Trzcinski2016-03-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | - Remove all view related methods from Ci::Build and CommitStatus - Remove unused Ci::Commit and Ci::Build methods - Use polymorphism to render different types of CommitStatus
* | | | Merge branch 'improve-award-emoji-spec' into 'master' Douwe Maan2016-03-141-4/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve award emoji test reliability and eliminate sleeps Closes #14129 See merge request !3195
| * | | | Improve award emoji test reliability and eliminate sleepsimprove-award-emoji-specStan Hu2016-03-131-4/+1
| | | | | | | | | | | | | | | | | | | | Closes #14129
* | | | | Merge branch 'share-project-ce' into 'master' Dmitriy Zaporozhets2016-03-146-0/+114
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring from EE: Share Project with Group - [x] Models and migrations - [x] Logic, UI - [x] Tests - [x] Documentation - [x] Share with group lock - [x] Api feature - [x] Api docs - [x] Api tests Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> For #12831 cc @DouweM @rspeicher @vsizov See merge request !3186
| * | | | | Bring shared project feature tests from EEDmitriy Zaporozhets2016-03-124-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Add finders logic and tests for shared projects featureDmitriy Zaporozhets2016-03-112-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | Merge branch 'issue-filter-tests' into 'master' Robert Speicher2016-03-142-18/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed tests for MR & issue filters See merge request !3203
| * | | | | | Fixed tests for MR & issue filtersissue-filter-testsPhil Hughes2016-03-142-18/+10
| | |_|/ / / | |/| | | |
* | | | | | Merge branch 'remove-sleeps-from-network-graph-spec' into 'master' Robert Speicher2016-03-142-5/+7
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove sleeps from network graph feature spec Also removed a double negative in the step description See merge request !3199
| * | | | | Remove sleeps from network graph feature specStan Hu2016-03-132-5/+7
| | |/ / / | |/| | |
* | | | | Use Capybara find methods and remove sleeps in feature specs in "All Issues" ↵improve-issues-filter-all-specStan Hu2016-03-131-8/+5
|/ / / / | | | | | | | | | | | | filter
* | | | Merge branch 'group-activity-separate-tab' into 'master' Robert Speicher2016-03-112-0/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move group activity feed to separate page for consistency with dashboard and project pages Fixes #14161. Part of #13480 See merge request !3157
| * | | | Move group activity feed to separate page for consistency with dashboard and ↵Dmitriy Zaporozhets2016-03-102-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | project pages Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Merge branch 'award-emoji-design-update' into 'master' Jacob Schatz2016-03-111-13/+13
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | Updated UI of award emoji Closes #13878 See merge request !3028
| * | | | Updated award emoji testsPhil Hughes2016-03-111-13/+13
| | | | |
* | | | | Merge branch 'account-settings' into 'master' Jacob Schatz2016-03-111-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Account settings Closes #13854 ![](https://gitlab.com/gitlab-org/gitlab-ce/uploads/4802a7b4f74f17d25a611ed1a2200445/account-settings.png) See merge request !3039
| * | | | | Fixed failing testsPhil Hughes2016-03-111-2/+2
| |/ / / /
* | | | | Merge branch 'comment-discard-button' into 'master' Jacob Schatz2016-03-117-12/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added discard button to comment form Also changed the labels on the buttons to better match the action they are completing. Closes #8057 See merge request !3085
| * | | | | Updated testsPhil Hughes2016-03-046-10/+10
| | | | | |
| * | | | | Fixed failing comment testPhil Hughes2016-03-041-1/+1
| | | | | |
| * | | | | Fixed tests for comment formsPhil Hughes2016-03-041-1/+1
| | |_|/ / | |/| | |
* | | | | Merge branch 'comment-error' into 'master' Dmitriy Zaporozhets2016-03-111-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add error for ajax:error when submitting comments Fixes #13814 ![Screen_Shot_2016-02-25_at_1.54.56_PM](/uploads/d70d706d306079df1b00d70f09affb63/Screen_Shot_2016-02-25_at_1.54.56_PM.png) cc @dblessing See merge request !2976
| * | | | Change test in an attempt to pass.comment-errorJacob Schatz2016-03-101-1/+1
| | | | | | | | | | | | | | | | | | | | Thanks @rspeicher!
* | | | | web hooks to webhooksashleys2016-03-102-4/+4
|/ / / /
* | | | Fixed failing testsPhil Hughes2016-03-104-14/+36
| |/ / |/| |
* | | Retry spinach testsKamil Trzcinski2016-03-092-0/+15
| | |