summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Use association search in issuable create serviceGrzegorz Bizon2016-04-221-6/+6
|
* Verify label affiliation before assigning to issueGrzegorz Bizon2016-04-221-2/+26
| | | | | | | 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 branch 'ci-commit-as-pipeline' into 'master' Kamil Trzciński2016-04-214-20/+32
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ci::Commit becomes a Pipeline object 1. Ci::Commit receives context: ref, :tag. 1. One Ci::Commit describes a one Pipeline 1. Pipeline is created from `.gitlab-ci.yml` 1. Pipeline is a ordered group of builds 1. We test MR against Pipeline 1. Pipelines have a separate view (https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3703) 1. Pipeline can be triggered from UI (https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3703) 1. Later we change `Trigger -> TriggerRequest -> Build` to `Trigger -> Pipeline` (future) 1. We add a Pipeline Hook that will be triggered on Pipeline status change (future) 1. We extend notifications to use `Pipeline Hook` to send summary on pipeline changes (future) After merging that I'll prepare a separate MR that will unify naming, database columns, table names: ``` Ci::Commit -> Pipeline Ci::Build -> Build CommitStatus -> Job GenericCommitStatus -> ExternalJob ci_commits -> pipelines ci_builds -> jobs ``` This MR implements first 5 points. This is made to solve this issue https://gitlab.com/gitlab-org/gitlab-ce/issues/14149. See merge request !3653
| * Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineci-commit-as-pipelineKamil Trzcinski2016-04-193-20/+27
| |\ | | | | | | | | | | | | # Conflicts: # db/schema.rb
| * \ Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineKamil Trzcinski2016-04-167-28/+43
| |\ \ | | | | | | | | | | | | | | | | # Conflicts: # db/schema.rb
| * \ \ Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineKamil Trzcinski2016-04-132-24/+36
| |\ \ \
| * | | | Fix rest of rspec and spinach testsKamil Trzcinski2016-04-121-1/+1
| | | | |
| * | | | Fix specsKamil Trzcinski2016-04-114-8/+10
| | | | |
| * | | | Use Ci::Commit as PipelineKamil Trzcinski2016-04-113-14/+24
| | | | |
* | | | | Fix wiki testsSebastian Klier2016-04-201-1/+1
| | | | |
* | | | | add wiki_page helper to url_builderSebastian Klier2016-04-201-1/+2
| | | | |
* | | | | add slack notifications for wiki pagesSebastian Klier2016-04-203-0/+50
| | | | | | | | | | | | | | | | | | | | update changelog
* | | | | Merge branch 'pmq20/gitlab-ce-issue_12785'Douwe Maan2016-04-193-43/+69
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Add support to cherry-pick any commitP.S.V.R2016-04-183-43/+69
| | |_|/ | |/| | | | | | | | | | | | | | Issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/12785 Merge Request: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3514
* | | | Using single builder for push and tag eventsGabriel Mazetto2016-04-192-2/+2
| | | |
* | | | Refactor GitTagPushService and fig tags_push system event hookGabriel Mazetto2016-04-191-14/+13
| | | |
* | | | Added System Hooks for push and tag_pushGabriel Mazetto2016-04-193-8/+16
|/ / / | | | | | | | | | | | | Code is based on Project Webhooks, removing deprecations and without commits listing.
* | | Memoize targetAlfredo Sumaran2016-04-161-22/+20
| | |
* | | Change variable nameAlfredo Sumaran2016-04-161-3/+3
| | |
* | | Fixes failing specAlfredo Sumaran2016-04-161-0/+2
| | |
* | | Update method name and remove unneeded paramsAlfredo Sumaran2016-04-161-2/+2
| | |
* | | Put owner and participating people firstAlfredo Sumaran2016-04-161-15/+24
| | |
* | | Merge branch 'issue_15155' into 'master' Rémy Coutable2016-04-141-2/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Setup visibility level for project when transfering for a group fixes #15155 See merge request !3707
| * | | Change transfer service to use existing methodsFelipe Artur2016-04-141-15/+3
| | | |
| * | | Setup visibility level for project when transfering for a groupFelipe Artur2016-04-131-4/+17
| | |/ | |/|
* | | Merge branch 'start-with-iid-on-new-branch' into 'master' Rémy Coutable2016-04-142-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Start with iid on branch creation After the discussion it in #3886 it was decided the iid should be in the beginning of the branch name. See merge request !3708
| * | | Start with iid on branch creationZeger-Jan van de Weg2016-04-142-2/+2
| | | |
* | | | Merge branch 'simplify-gitlab-url_builder-15202' into 'master' Rémy Coutable2016-04-142-3/+2
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactor and expose only Gitlab::UrlBuilder.build(record) ``` $ git grep Gitlab::UrlBuilder app/models/commit.rb: url: Gitlab::UrlBuilder.build(self), app/services/issues/base_service.rb: issue_url = Gitlab::UrlBuilder.build(issue) app/services/merge_requests/base_service.rb: hook_data[:object_attributes][:url] = Gitlab::UrlBuilder.build(merge_request) app/views/search/results/_note.html.haml:- note_url = Gitlab::UrlBuilder.build(note) lib/gitlab/note_data_builder.rb: base_data[:object_attributes][:url] = Gitlab::UrlBuilder.build(note) spec/lib/gitlab/note_data_builder_spec.rb: expect(data[:object_attributes][:url]).to eq(Gitlab::UrlBuilder.build(note)) spec/lib/gitlab/url_builder_spec.rb:describe Gitlab::UrlBuilder, lib: true do ``` Fixes #15202. See merge request !3696
| * | | Refactor and expose only Gitlab::UrlBuilder.build(record)simplify-gitlab-url_builder-15202Rémy Coutable2016-04-132-3/+2
| |/ / | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Instrument the HousekeepingService classinstrument-house-keepingYorick Peterse2016-04-131-4/+10
|/ / | | | | | | | | This allows us to track how much time is spent in updating the "pushes_since_gc" column as well as the time needed to obtain the lease.
* | Merge branch 'decouple-member-notification' into 'master' Douwe Maan2016-04-121-24/+28
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-088-36/+83
| |\ \ | | |/
| * | Remove useless Notification modelDmitriy Zaporozhets2016-03-301-9/+15
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Refactor notification helper and fix notification serviceDmitriy Zaporozhets2016-03-301-1/+1
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Update NotificationService to use NotificationSettings instead of membershipDmitriy Zaporozhets2016-03-301-18/+16
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Delete notes via APIRobert Schilling2016-04-121-0/+8
| |/ |/|
* | Merge branch 'issue_14012' into 'master' Rémy Coutable2016-04-071-1/+1
|\ \ | | | | | | | | | | | | | | | | | | Fix problem when creating milestones in groups without projects Fixes #14012 See merge request !3481
| * | Implement review suggestionsFelipe Artur2016-04-071-1/+1
| | |
* | | Only update main language if it is not already setupdate_main_lang_if_unsetDrew Blessing2016-04-061-5/+5
|/ /
* | Merge branch 'reorder-language' into 'master' Yorick Peterse2016-04-051-3/+7
|\ \ | | | | | | | | | | | | Update language after doing all other operations See merge request !3533
| * | Update language only on HEAD of the repositoryreorder-languageKamil Trzcinski2016-04-051-0/+3
| | |
| * | Update language after doing all other operationsKamil Trzcinski2016-04-051-3/+4
| |/
* | Flush repository cache before import project datafix-gh-pr-importDouglas Barbosa Alexandre2016-04-041-0/+2
| | | | | | | | | | 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/+19
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 incorrect variable nameBaldinof2016-04-041-1/+1
| | |
| * | Move unlink fork logic to a serviceBaldinof2016-03-221-0/+19
| | |
* | | Merge branch 'routing' into 'master' Robert Speicher2016-04-011-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Added & use Gitlab::Routing for URL helpers Extracted from !3389 See merge request !3486
| * | | Added & use Gitlab::Routing for URL helpersroutingYorick Peterse2016-04-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails' "url_helpers" method creates an anonymous Module (which a bunch of methods) on every call. By caching the output of this method in a dedicated method we can shave off about 10 seconds of loading time for an issue with around 200 comments.
* | | | Merge branch ↵Douwe Maan2016-04-011-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'snippets-with-comments-cause-a-500-when-they-show-up-in-search-results-14764' into 'master' Fix Error 500 when searching for a comment in a project snippet Closes #14764. /cc @stanhu, and thank you for the spec! ;) See merge request !3468