summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'james11/gitlab-ce-removable-group-owner' into 'master' Robert Speicher2015-11-1714-77/+149
|\ | | | | | | | | | | | | | | | | | | Prevent the last owner of a group from being able to delete themselves by 'adding' themselves as a master Replaces !1708. Fixes #1111. See merge request !1815
| * Add changelog itemjames11/gitlab-ce-removable-group-ownerDouwe Maan2015-11-171-0/+1
| |
| * Use project member abilities more extensivelyDouwe Maan2015-11-177-45/+49
| |
| * Refactor ability changesDouwe Maan2015-11-175-71/+67
| |
| * Merge branch 'master' into james11/gitlab-ce-removable-group-ownerDouwe Maan2015-11-17495-2349/+8219
| |\
| * | fixing rubocop - random code not related to the changesJames Lopez2015-11-121-5/+9
| | |
| * | fixing rubocop indentsJames Lopez2015-11-121-1/+2
| | |
| * | fixing rubocop indentsJames Lopez2015-11-122-12/+14
| | |
| * | updated exceptionJames Lopez2015-11-111-1/+1
| | |
| * | refactored code as projects only have one owner. Kept some refactoring in ↵James Lopez2015-11-114-40/+35
| | | | | | | | | | | | place (has_owners concern)
| * | refactored permissions and added update_project_member ability logic. Also ↵James Lopez2015-11-035-24/+57
| | | | | | | | | | | | refactored owner methods to a concern.
| * | fix issue with adding members to project (spotted by test)James Lopez2015-10-301-2/+11
| | |
| * | fixed last group owner issue and added testJames Lopez2015-10-293-4/+31
| | |
* | | Merge branch 'remove-code-duplication'Dmitriy Zaporozhets2015-11-1722-434/+371
|\ \ \ | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Code duplication check should be enabled nowremove-code-duplicationDmitriy Zaporozhets2015-11-171-1/+0
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove small code duplication in user_reference_filter.rbDmitriy Zaporozhets2015-11-171-6/+9
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Refactor similar code for Issue and MR update serviceDmitriy Zaporozhets2015-11-173-57/+53
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove code duplication in notification_service.rbDmitriy Zaporozhets2015-11-171-28/+18
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove duplication in mailers/emails/notes.rbDmitriy Zaporozhets2015-11-172-45/+52
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Set higher flay value to avoid unnecessary refactoring for nowDmitriy Zaporozhets2015-11-171-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Refactor duplciate code for groups_controller.rb and ↵Dmitriy Zaporozhets2015-11-176-50/+46
| | | | | | | | | | | | | | | | | | | | | | | | slack_service/note_message.rb Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove duplication in diff_helper.rbDmitriy Zaporozhets2015-11-171-16/+16
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove code duplication in gitlab_markdown_helper.rbDmitriy Zaporozhets2015-11-171-28/+24
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove duplication in issue emailsDmitriy Zaporozhets2015-11-161-36/+30
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Allow flay to fail for now since there is still a lot of refactoring todoDmitriy Zaporozhets2015-11-161-0/+1
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Remove duplication in reference filtersDmitriy Zaporozhets2015-11-164-166/+119
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Refactor select2 tagsDmitriy Zaporozhets2015-11-162-11/+14
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Dont allow code duplication check to failDmitriy Zaporozhets2015-11-161-1/+0
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | Bump VERSIONRobert Speicher2015-11-171-1/+1
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge branch 'param-view' into 'master'Robert Speicher2015-11-176-9/+12
|\ \ \ \ | | | | | | | | | | | | | | | See merge request !1816
| * | | | Don't use params[:view] directly.param-viewDouwe Maan2015-11-176-9/+12
| | |_|/ | |/| |
* | | | Update CHANGELOGRobert Speicher2015-11-171-1/+3
| | | | | | | | | | | | | | | | [ci skip]
* | | | Update monthly release templateRobert Speicher2015-11-171-12/+16
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #9833 from huacnlee/fix/pr-9820-missingsRobert Speicher2015-11-171-2/+1
|\ \ \ \ | | | | | | | | | | Don't render note edit forms for visitors
| * | | | Avoid render edit_form when visitor can't edit them.Jason Lee2015-11-171-2/+1
|/ / / / | | | | | | | | | | | | Reverted #9820, github/task_list need a form, textarea for update.
* | | | Merge branch 'rs-revert-gh-9820' into 'master' Robert Speicher2015-11-179-18/+24
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Merge pull request #9820 from huacnlee/avoid-render-form-in-notes-list" This reverts commit 63144cd062f6d259f1f30b6e06eb92a16caa8dec, reversing changes made to 8ab5df9d872414b2cca3ebd16d57b89e2f19e06a. Reverts https://github.com/gitlabhq/gitlabhq/pull/9820 See merge request !1804
| * | | Revert "Merge pull request #9820 from huacnlee/avoid-render-form-in-notes-list"rs-revert-gh-9820Robert Speicher2015-11-169-18/+24
| | | | | | | | | | | | | | | | | | | | This reverts commit 63144cd062f6d259f1f30b6e06eb92a16caa8dec, reversing changes made to 8ab5df9d872414b2cca3ebd16d57b89e2f19e06a.
* | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDmitriy Zaporozhets2015-11-173-1/+29
|\ \ \ \
| * \ \ \ Merge branch 'issue/vars-need-eval' into 'master' Douwe Maan2015-11-171-1/+1
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensure variable in mail is evalled I received this snippet in the mail: ``` Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Project #{@old_path_with_namespace} was moved to another location The project is now located under ``` This MR fixes it. Notes: * I did a quick grep on other 'text.erb' files, but this was the only one with this type of error * I made no changes to the specs. The only [spec](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/spec/mailers/notify_spec.rb#L412) I could find passes because (I assume) the correct value is in the html-part See merge request !1706
| | * | | | Ensure variable is evalledGert Goet2015-10-291-1/+1
| | | | | |
| * | | | | Merge branch 'rs-wiki-touches-project' into 'master' Douwe Maan2015-11-172-0/+28
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make ProjectWiki touch Project#last_activity_at after wiki actions Closes #3026 See merge request !1803
| | * | | | | Make ProjectWiki touch Project#last_activity_at after wiki actionsrs-wiki-touches-projectRobert Speicher2015-11-162-0/+28
| | | |/ / / | | |/| | | | | | | | | | | | | | | Closes #3026
* | | | | | Merge pull request #9837 from Great-Antique/masterRobert Schilling2015-11-171-2/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Fix md syntax in doc/api/commits.md
| * | | | | Fix md syntax in doc/api/commits.mdDen Girnyk2015-11-171-2/+2
|/ / / / /
* | | | | Merge branch 'http-to-https' into 'master' Achilleas Pipinellis2015-11-1720-45/+45
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | Rewrite HTTP links to force TLS, where possible I got annoyed at the fact that the links on the profile page don't force the use of TLS, so I grepped through the entire source tree, tested all the links I found, and replaced them if possible. See merge request !1806
| * | | | Rewrite HTTP links to force TLS, where possibleAlex Jordan2015-11-1620-45/+45
| | | | |
* | | | | Merge branch 'global-milestones' into 'master' Dmitriy Zaporozhets2015-11-1637-258/+398
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create milestones in the group When you work with groups its quite often you want to create same milestone in multiple projects. This MR allows you to do so For #3488 Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> See merge request !1797
| * | | | | Some code and doc improvementsglobal-milestonesDmitriy Zaporozhets2015-11-164-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Fix removing avatar for groupDmitriy Zaporozhets2015-11-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Group masters should be able to create/close milestonesDmitriy Zaporozhets2015-11-165-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>