summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Add support for Webhook note eventsStan Hu2015-05-211-1/+1
| | | | Closes https://github.com/gitlabhq/gitlabhq/issues/6745
* Fix.Douwe Maan2015-05-151-1/+1
|
* Move stuff around a bit in NotifictionService.Douwe Maan2015-05-151-8/+15
|
* Call merge request web hook for each commitsArthur Gautier2015-05-131-0/+15
| | | | | | | Call merge request web hook every times a new commit hits the source branch Signed-off-by: Arthur Gautier <baloo@gandi.net>
* Refactor SystemNoteService.cross_reference_disallowed?Robert Speicher2015-05-111-6/+13
|
* Update SystemNoteService method naming conventionsRobert Speicher2015-05-116-77/+73
| | | | | Now the verb comes first, and there is no restriction on singular/plural.
* Remove legacy special case for emphasized reference notesRobert Speicher2015-05-111-13/+9
|
* Add a SystemNoteService classRobert Speicher2015-05-116-8/+313
| | | | | There's a lot of code in the Note model that only deals with creating system notes, so we're going to split that into its own class.
* Added X-GitLab-Event header for web hooksbugagazavr2015-05-082-4/+4
|
* Make the first branch pushed to an empty repository the default HEAD.Stan Hu2015-05-051-0/+4
| | | | | | | | | | | In an empty repository, pushing a new branch not called "master" would leave HEAD in an unknown state, causing ambiguity if another branch were pushed. This could in turn cause a new protected branch to be created and cause the default branch to change. * Closes #1561 * Closes #1576 * Closes https://github.com/gitlabhq/gitlabhq/issues/8883
* Revert "Added X-GitLab-Event header for web hooks"Valery Sizov2015-04-272-4/+4
| | | | This reverts commit 548f182814acd0f7a110e6c165c186e345901b00.
* Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDmitriy Zaporozhets2015-04-272-53/+21
|\
| * Use Projects::CreateService to fork projects so that after-create hooks are run.forked-eventDouwe Maan2015-04-242-53/+21
| |
* | Added X-GitLab-Event header for web hooksbugagazavr2015-04-252-4/+4
|/
* Link cross-project cross-reference notes to correct project.Douwe Maan2015-04-243-4/+3
|
* No longer needed to pass project argument to commit methods.Douwe Maan2015-04-243-13/+9
|
* Use project.commit convenience method.Douwe Maan2015-04-243-3/+3
|
* Let commit model know about its project.Douwe Maan2015-04-241-1/+1
|
* Clean up code somewhat.better-commit-mentionsDouwe Maan2015-04-171-13/+15
|
* Only send note notifications to people accessible by the note author.Douwe Maan2015-04-151-2/+2
| | | | Should not make a difference most of the time, since if they're participating in the thread, they have access to the project.
* Clean up code around commit mentions.Douwe Maan2015-04-152-10/+5
|
* Fix for snippets.fix-mention-notificationDouwe Maan2015-04-151-2/+4
|
* Fix Mention notification level.Douwe Maan2015-04-151-24/+13
|
* Let invites be declined.Douwe Maan2015-04-141-0/+8
|
* Add emails around invitation.Douwe Maan2015-04-141-0/+16
|
* Track who created a group or project member.Douwe Maan2015-04-142-2/+2
|
* Merge branch 'reference-access-control' into 'master'Dmitriy Zaporozhets2015-04-133-29/+33
|\ | | | | | | | | | | | | | | Only allow users to reference groups, projects, issues, MRs, commits they have access to. Addresses https://dev.gitlab.org/gitlab/gitlabhq/issues/2183. See merge request !1742
| * Fix errors.Douwe Maan2015-04-021-12/+7
| |
| * Only allow users to cross-reference and close issues they have access to.Douwe Maan2015-04-021-2/+2
| |
| * Only allow user to see participants from groups they have access to.Douwe Maan2015-04-021-2/+2
| |
| * Add Commit#author and #committer.Douwe Maan2015-04-021-1/+1
| |
| * Slightly refactor getting note notification recipients.Douwe Maan2015-04-021-14/+23
| |
* | Merge branch 'regex-start-of-string' into 'master'Dmitriy Zaporozhets2015-04-121-3/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | Fix persistent XSS vulnerability around profile website URLs. Fixes gitlab/gitlab-ee#268 See merge request !1761
| * | Use `\A` and `\z` in regexes instead of `^` and `$`.Douwe Maan2015-04-101-3/+1
| | |
* | | Merge branch 'ci_fork' into 'master'Dmitriy Zaporozhets2015-04-091-0/+6
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | Project fork on CI side https://dev.gitlab.org/gitlab/gitlab-ci/issues/187 !!! GitLab CI side MR - https://gitlab.com/gitlab-org/gitlab-ci/merge_requests/56 See merge request !499
| * | CI forking: testsValery Sizov2015-04-061-1/+1
| | |
| * | CI project forkingValery Sizov2015-04-061-0/+6
| | |
* | | Merge branch 'configurable-attachment-size' into 'master'Douwe Maan2015-04-031-1/+7
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support configurable attachment size in Application Settings page ### What does this MR do? This MR provides the ability to configure the maximum size of an attachment inside a note. A parameter has been added to the Application Settings page. ### Are there points in the code the reviewer needs to double check? What should be done with the legacy note attachment validation? I added code to make the validation work with the configurable setting. I could see an issue where an admin lowers the limit from 10 megabytes to 5 megabytes, which could cause an existing model to be invalid. ### Why was this MR needed? We often have attachments that exceed 10 MB, and it would be nice to be able to override the defaults. ### What are the relevant issue numbers / [Feature requests](http://feedback.gitlab.com/)? See Issue #1258 ### Screenshots Before: ![Screen_Shot_2015-03-29_at_3.06.53_PM](https://gitlab.com/gitlab-org/gitlab-ce/uploads/6013a1dbc8cf61a63e93744149937fa0/Screen_Shot_2015-03-29_at_3.06.53_PM.png) After: ![Screen_Shot_2015-03-29_at_3.12.34_PM](https://gitlab.com/gitlab-org/gitlab-ce/uploads/f3518af7e8653ba40f0a3579456da6ad/Screen_Shot_2015-03-29_at_3.12.34_PM.png) See merge request !407
| * | Support configurable attachment size via Application SettingsStan Hu2015-04-021-1/+7
| |/ | | | | | | | | | | Fix bug where error messages from Dropzone would not be displayed on the issues page Closes #1258
* | Merge branch 'repository-archive-worker' into 'master'Dmitriy Zaporozhets2015-04-021-8/+56
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Archive repositories in background worker. Depends on https://gitlab.com/gitlab-org/gitlab_git/merge_requests/17 being merged, a new `gitlab_git` being released and this MR's `Gemfile.lock` being updated.. See private issue https://dev.gitlab.org/gitlab/gitlabhq/issues/2173. To do after this is merged: Update https://gitlab.com/gitlab-org/omnibus-gitlab/blob/master/files/gitlab-cookbooks/gitlab/templates/default/sv-sidekiq-run.erb in omnibus. See merge request !436
| * | Add tests.Douwe Maan2015-03-311-2/+4
| | |
| * | Small code cleanup.Douwe Maan2015-03-311-9/+4
| | |
| * | Archive repositories in background worker.Douwe Maan2015-03-311-8/+59
| |/
* | Use more specific regexes.Douwe Maan2015-03-271-2/+2
|/
* No magic numbers for issues filteringDmitriy Zaporozhets2015-03-262-4/+4
|
* notification on project movingnotification_on_project_movingValery Sizov2015-03-251-0/+3
|
* Merge branch 'api-internal-errors' into 'master'Dmitriy Zaporozhets2015-03-253-3/+3
|\ | | | | | | | | | | | | | | | | | | Respond with full GitAccess error if user has project read access. Should help with debugging #1236. cc @marin See merge request !437
| * Refactor GitAccess to use instance variables.Douwe Maan2015-03-243-3/+3
| |
* | Merge branch 'unset-assignee' into 'master'Dmitriy Zaporozhets2015-03-243-3/+9
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure issue assignee is properly reset. Previously, when the assignee was reset via the sidebar or bulk edit, `assignee_id` was set to `-1` rather than `null`, which caused the two issues shown below: ![Screen_Shot_2015-03-24_at_16.52.13](https://gitlab.com/gitlab-org/gitlab-ce/uploads/3c937795c45031c3c72c124ced866598/Screen_Shot_2015-03-24_at_16.52.13.png) - A "(deleted)" participant - An empty selectbox in the sidebar, instead of "Select assignee" See merge request !443
| * | Make sure issue assignee is properly reset.Douwe Maan2015-03-243-3/+9
| |/
View Lorry Controller Status