summaryrefslogtreecommitdiff
path: root/spec/services
Commit message (Collapse)AuthorAgeFilesLines
* Use association search in issuable create serviceGrzegorz Bizon2016-04-221-6/+6
|
* Add affinity checks in issue create serviceGrzegorz Bizon2016-04-221-2/+9
|
* Update specs to accomodate issuable assign changesGrzegorz Bizon2016-04-224-21/+31
|
* Verify label affiliation before assigning to issueGrzegorz Bizon2016-04-221-0/+28
| | | | | | | This also verify if milestone belongs to correct project before creating a new issue. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/15439
* Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineci-commit-as-pipelineKamil Trzcinski2016-04-191-12/+12
|\ | | | | | | | | # Conflicts: # db/schema.rb
| * Refactor GitTagPushService and fig tags_push system event hookGabriel Mazetto2016-04-191-12/+12
| |
* | Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineKamil Trzcinski2016-04-162-10/+24
|\ \ | |/ | | | | | | # Conflicts: # db/schema.rb
| * Delete tags via ruggedRobert Schilling2016-04-151-10/+1
| |
| * Setup visibility level for project when transfering for a groupFelipe Artur2016-04-131-0/+23
| |
* | Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineKamil Trzcinski2016-04-132-16/+28
|\ \ | |/
| * Merge branch 'decouple-member-notification' into 'master' Douwe Maan2016-04-121-16/+13
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decouple membership and notifications This allow you to have notification setting per project even if you are member of group. It also creates background for having notification settings in project you are not member of. - [x] Make it work - [x] Migrations - [x] CHANGELOG - [x] More tests - [x] API For #3359 After this merge request there is still some work to be done: * create migration that remove duplicates in notification settings table and create uniq index (8.8 probably) * remove notification_level field from Member model in 9.0 * make proper API for notification settings * use `MemberCreateService` instead of Member#after_create callback for creating notification settings (after #14709) * maybe more tests Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> See merge request !3421
| | * Reuse `User#notification_settings_for` when it's possibleDouglas Barbosa Alexandre2016-04-111-2/+2
| | |
| | * Merge branch 'master' into decouple-member-notificationDouglas Barbosa Alexandre2016-04-085-26/+150
| | |\
| | * | Small refactoring and cleanup of notification logicDmitriy Zaporozhets2016-03-301-6/+3
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * | Update NotificationService to use NotificationSettings instead of membershipDmitriy Zaporozhets2016-03-301-13/+13
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Adapt tests to new testing guidelinesRobert Schilling2016-04-121-5/+4
| | | |
| * | | Fix code review issuesRobert Schilling2016-04-121-4/+5
| | | |
| * | | Delete notes via APIRobert Schilling2016-04-121-0/+15
| | |/ | |/|
* | | Fix specsKamil Trzcinski2016-04-112-3/+3
|/ /
* | Only update main language if it is not already setupdate_main_lang_if_unsetDrew Blessing2016-04-061-5/+21
| |
* | Merge branch 'reorder-language' into 'master' Yorick Peterse2016-04-051-4/+14
|\ \ | | | | | | | | | | | | Update language after doing all other operations See merge request !3533
| * | Update language only on HEAD of the repositoryreorder-languageKamil Trzcinski2016-04-051-4/+14
| |/
* | Flush repository cache before import project datafix-gh-pr-importDouglas Barbosa Alexandre2016-04-041-0/+17
| | | | | | | | | | GitHub Pull Requests importer handle with the repository while importing data, we need to make sure that the cached values are valid.
* | Merge branch 'fix_remove_fork_link' into 'master' Douwe Maan2016-04-041-0/+32
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove fork link closes all merge requests opened on source project Currently, if you: - create a fork - open a merge request on the source project - remove the fork link The created MR cannot be closed. With this MR, all pending MR is closed when the fork link is removed. See merge request !3189
| * | Fix rubocop in unlink fork service specsBaldinof2016-04-031-2/+2
| | |
| * | Move unlink fork logic to a serviceBaldinof2016-03-221-0/+32
| | |
* | | Merge branch 'fix-mentions-on-confidential-issues-for-non-members' into ↵Douwe Maan2016-04-011-21/+56
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Mentions on confidential issues doesn't create todos for non-members Closes #14569 See merge request !3374
| * | | Mentions on confidential issues doesn't create todos for non-membersfix-mentions-on-confidential-issues-for-non-membersDouglas Barbosa Alexandre2016-03-301-21/+56
| | |/ | |/|
* | | Improve specs for issue move serviceGrzegorz Bizon2016-03-311-2/+2
| | |
* | | Refactor uploads rewriter used when moving issueGrzegorz Bizon2016-03-301-7/+2
| | |
* | | Add uploads rewriter and use it when moving issueGrzegorz Bizon2016-03-301-0/+19
|/ /
* | Preserve time notes has been updated at when moving issueGrzegorz Bizon2016-03-231-2/+19
| |
* | Merge branch 'notifications-for-subscribers-confidential-issue-labels' into ↵Douwe Maan2016-03-232-1/+94
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Restrict notifications for confidential issues Closes #14468 /cc @rymai See merge request !3334
| * | Restrict notifications for confidential issuesDouglas Barbosa Alexandre2016-03-222-1/+94
| | |
* | | Do not allow to move issue if it has not been persistedmove-issue-section-should-not-be-displayed-in-the-new-issue-form-14489Grzegorz Bizon2016-03-231-0/+6
|/ /
* | Merge branch 'master' into issue_12658Douwe Maan2016-03-212-0/+266
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/models/issue.rb # app/views/projects/_home_panel.html.haml # app/views/shared/projects/_project.html.haml # db/schema.rb # spec/models/project_spec.rb
| * | Preserve created at time of notes when moving issueGrzegorz Bizon2016-03-201-0/+5
| | |
| * | Do not pass params that are not used in issue move serviceGrzegorz Bizon2016-03-201-2/+1
| | |
| * | Merge branch 'master' into feature/issue-moveGrzegorz Bizon2016-03-195-7/+110
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (121 commits) Dedupe labels in labels selector in Dashboard pages Refactor colors and lists Add a safeguard in MergeRequest#compute_diverged_commits_count Fix an issue when the target branch of a MR had been deleted Add avatar to issue and MR pages header Cleanup somce css colors Re-group scss variables Refactor `Todo#target` Fixes issue with filter label missing on labels & milestones Rename `Todo#to_reference` to `Todo#target_reference` Fixed failing tests Updated controller with before_action Fixed other issues based on feedback Fixes issue on dashboard issues Full labels data in JSON Fixed issue with labels dropdown getting wrong labels Update CHANGELOG Use `Note#for_project_snippet?` to skip notes on project snippet Use `Commit#short_id` instead of `Commit.truncate_sha` Reuse `for_commit?` on conditional validations Update schema info comment on todo related files ... Conflicts: app/models/issue.rb db/schema.rb spec/models/issue_spec.rb
| * | | Improvements in issue move feaure (refactoring)Grzegorz Bizon2016-03-191-53/+19
| | | | | | | | | | | | | | | | According to endbosses' suggestions.
| * | | Prevent issue move if issue has been already movedGrzegorz Bizon2016-03-171-0/+18
| | | |
| * | | Add minor improvements in code related to issue moveGrzegorz Bizon2016-03-171-1/+1
| | | |
| * | | Use helper to create list of projects issue can be moved toGrzegorz Bizon2016-03-171-2/+13
| | | | | | | | | | | | | | | | This also adds confirmation message if issue move has been requested.
| * | | Add new notifications for issue move actionGrzegorz Bizon2016-03-171-1/+1
| | | | | | | | | | | | | | | | [ci skip]
| * | | Reuse existing issue services when moving issueGrzegorz Bizon2016-03-171-1/+1
| | | |
| * | | Make it possible to move issue if user is a reporterGrzegorz Bizon2016-03-171-10/+10
| | | | | | | | | | | | | | | | | | | | Discussed it here: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2831#note_4190228
| * | | Preserve original author when moving issueGrzegorz Bizon2016-03-171-4/+4
| | | | | | | | | | | | | | | | | | | | This also wrapps entire process into transation, as rewriting references may have large memory footprint.
| * | | Add reference unfold pipeline used when moving issueGrzegorz Bizon2016-03-171-2/+2
| | | |
| * | | Minor refactoring of issue move service and specsGrzegorz Bizon2016-03-171-2/+3
| | | |
| * | | Rewrite references in notes when moving issueGrzegorz Bizon2016-03-171-15/+42
| | | |