summaryrefslogtreecommitdiff
path: root/app/models
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'fix-admin-should-be-able-to-add-himself-to-group' into 'master' Dmitriy Zaporozhets2015-12-091-8/+4
|\ | | | | | | | | | | | | | | | | Signed in Admin can add/remove himself to a group/project Fixes #3640 /cc @JobV Could you have a look as well? See merge request !1942
| * Merge branch 'master' into fix-admin-should-be-able-to-add-himself-to-groupDouglas Barbosa Alexandre2015-12-0823-149/+310
| |\
| * \ Merge branch 'master' into fix-admin-should-be-able-to-add-himself-to-groupDouglas Barbosa Alexandre2015-12-046-20/+15
| |\ \
| * \ \ Merge branch 'master' into fix-admin-should-be-able-to-add-himself-to-groupDouglas Barbosa Alexandre2015-12-033-3/+8
| |\ \ \
| * | | | Signed in admin should be able to add/remove himself to a projectDouglas Barbosa Alexandre2015-12-011-4/+2
| | | | |
| * | | | Signed in admin should be able to add/remove himself to a groupDouglas Barbosa Alexandre2015-12-011-4/+2
| | | | |
* | | | | Merge branch 'master' into merge-if-greenDouwe Maan2015-12-096-0/+37
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/controllers/projects/merge_requests_controller.rb # config/routes.rb
| * | | | | Update annotationsStan Hu2015-12-085-0/+26
| | | | | |
| * | | | | Merge branch 'edit-in-patch-branch' into 'master' Robert Speicher2015-12-081-0/+11
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default target branch to patch-n when editing file in protected branch Fixes #3441 See merge request !2021
| | * | | | | Default target branch to patch-n when editing file in protected branchDouwe Maan2015-12-081-0/+11
| | | | | | |
* | | | | | | Merge branch 'master' into merge-if-greenDouwe Maan2015-12-081-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/views/projects/merge_requests/widget/_heading.html.haml # app/views/projects/merge_requests/widget/open/_accept.html.haml
| * | | | | | Merge branch 'build-related-fixes' into 'master' Dmitriy Zaporozhets2015-12-081-1/+1
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Build related UI fixes ## Before ![Screen_Shot_2015-12-08_at_13.15.21](/uploads/b72ed55da1f0ec953d2eb077fa4fbb18/Screen_Shot_2015-12-08_at_13.15.21.png) ## After ![Screen_Shot_2015-12-08_at_13.15.01](/uploads/144d8099e2724a51125e392dcbd43ed4/Screen_Shot_2015-12-08_at_13.15.01.png) See merge request !2015
| | * | | | | Memoize ci_yaml_file.Douwe Maan2015-12-081-1/+1
| | | |_|_|/ | | |/| | |
* | | | | | Merge branch 'master' into zj/gitlab-ce-merge-if-greenDouwe Maan2015-12-081-0/+12
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge branch 'serve_lfs_object' into 'master' Douwe Maan2015-12-081-0/+12
| |\ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Serve LFS object Depends on gitlab-org/gitlab_git!57 See merge request !1976
| | * | | | Add specs for showing lfs object in UI.Marin Jankovski2015-12-071-0/+4
| | | | | |
| | * | | | Recursivity needed if a fork is a fork of a fork.Marin Jankovski2015-12-041-1/+1
| | | | | |
| | * | | | Don't show diffs for lfs pointers.Marin Jankovski2015-12-041-2/+0
| | | | | |
| | * | | | If a user clicks on the LFS object, it should be served if the user has ↵Marin Jankovski2015-12-031-0/+10
| | | |/ / | | |/| | | | | | | | | | | | access to the object.
* | | | | Merge branch 'master' into zj/gitlab-ce-merge-if-greenDouwe Maan2015-12-0818-106/+179
|\ \ \ \ \ | |/ / / /
| * | | | Add custom NamespaceNameValidatorRobert Speicher2015-12-071-3/+3
| | | | |
| * | | | Add custom NamespaceValidatorRobert Speicher2015-12-072-9/+5
| | | | |
| * | | | Add custom LineCodeValidatorRobert Speicher2015-12-072-2/+2
| | | | |
| * | | | Add custom ColorValidatorRobert Speicher2015-12-072-7/+5
| | | | |
| * | | | Add custom UrlValidatorRobert Speicher2015-12-078-37/+27
| | | | |
| * | | | Merge branch 'fix-parallel-merge' into 'master' Dmitriy Zaporozhets2015-12-071-2/+2
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dont use cached collection for Repository find_branch and find_tag methods Fix for #3816 Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> See merge request !2003
| | * | | | Dont use cached collection for Repository find_branch and find_tag methodsDmitriy Zaporozhets2015-12-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Merge branch 'report-ssl-errors' of https://gitlab.com/stanhu/gitlab-ceDmitriy Zaporozhets2015-12-071-17/+19
| |\ \ \ \ \ | | |/ / / / | |/| | | |
| | * | | | Handle and report SSL errors in Web hook test. Check for status 200 for success.Stan Hu2015-12-041-17/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a Web hook test fails due to an SSL error or some other error, report the result back to the user instead of an Error 500. Closes #3656 Handle response
| * | | | | Merge branch 'webhook_payload_with_changes' into 'master' Valery Sizov2015-12-072-3/+27
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add added, modified and removed properties to commit object in webhook https://gitlab.com/gitlab-org/gitlab-ee/issues/20 See merge request !1988
| | * | | | | fixes after reviewwebhook_payload_with_changesValery Sizov2015-12-071-16/+9
| | | | | | |
| | * | | | | fox specsValery Sizov2015-12-072-15/+24
| | | | | | |
| | * | | | | Add added, modified and removed properties to commit object in webhookValery Sizov2015-12-041-1/+23
| | |/ / / /
| * | | | | Merge branch 'gsmethells/gitlab-ce-sort-by-due-date'Douwe Maan2015-12-071-0/+29
| |\ \ \ \ \
| | * | | | | Satisfy RubocopDouwe Maan2015-12-071-1/+1
| | | | | | |
| | * | | | | Merge branch 'master' into gsmethells/gitlab-ce-sort-by-due-dategsmethells/gitlab-ce-sort-by-due-dateDouwe Maan2015-12-0713-63/+146
| | |\ \ \ \ \
| | * | | | | | sort milestones by due_dateGreg Smethells2015-12-031-1/+30
| | | |_|/ / / | | |/| | | |
| * | | | | | Merge branch 'fix/award-emoji-conflict-in-notes' into 'master' Grzegorz Bizon2015-12-071-0/+30
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix problems with award-emoji-only comment This fixes a conflict between note with only a single emoji in content and award-emojis mechanisms. Closes #3734 cc @vsizov See merge request !1936
| | * | | | | | Simplify `contains_emoji_only?` method in `Note`fix/award-emoji-conflict-in-notesGrzegorz Bizon2015-12-071-2/+1
| | | | | | | |
| | * | | | | | Refactor note awards to reuse `emoji_pattern` and improve validatorGrzegorz Bizon2015-12-051-18/+14
| | | | | | | |
| | * | | | | | Make method `supports_award?` public in `Note`Grzegorz Bizon2015-12-031-2/+2
| | | | | | | |
| | * | | | | | Support emoji awards also in merge requestsGrzegorz Bizon2015-12-031-3/+4
| | | | | | | |
| | * | | | | | Move note emoji-award implementation to note model (feature envy)Grzegorz Bizon2015-12-031-0/+33
| | | | | | | |
| | * | | | | | Add validator for award-emoji noteGrzegorz Bizon2015-12-031-0/+1
| | |/ / / / /
| * | | | | | Merge branch 'fix-global-milestones-error-500' into 'master' Douwe Maan2015-12-071-1/+1
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Error 500 when creating global milestones with Unicode characters Two issues: 1. The constraints in the resources were incorrect. Here's what it was before: ``` group_milestone GET /groups/:group_id/milestones/:id(.:format) groups/milestones#show {:id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/} ``` In this case, id is actually the title of the milestone, which can be anything at the moment. After: ``` group_milestone GET /groups/:group_id/milestones/:id(.:format) groups/milestones#show {:id=>/[^\/]+/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/} ``` 2. `parameterize` would strip all Unicode characters, leaving a blank string. Rails would report something like: ``` ActionView::Template::Error (No route matches {:action=>"show", :controller=>"groups/milestones", :group_id=>#<Group id: 48, name: "ops-dev", path: "ops-dev", owner_id: nil, created_at: "2015-11-15 08:55:30", updated_at: "2015-12-02 06:23:26", type: "Group", description: "", avatar: "sha1.c71e73d51af1865c1bbbf6208e10044d46c9bb93.png", public: false>, :id=>"", :title=>"肯定不是中文的问题"} missing required keys: [:id]): ``` This change uses the babosa library to create a better slug, which surprisingly isn't actually used by the global milestone controllers. Instead, they use the title passed as a query string for some reason. Closes https://github.com/gitlabhq/gitlabhq/issues/9881 See merge request !1983
| | * | | | | | Fix Error 500 when creating global milestones with Unicode charactersStan Hu2015-12-051-1/+1
| | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two issues: 1. The constraints in the resources were incorrect. Here's what it was before: ``` group_milestone GET /groups/:group_id/milestones/:id(.:format) groups/milestones#show {:id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/} ``` In this case, id is actually the title of the milestone, which can be anything at the moment. After: ``` group_milestone GET /groups/:group_id/milestones/:id(.:format) groups/milestones#show {:id=>/[^\/]+/, :group_id=>/[a-zA-Z.0-9_\-]+(?<!\.atom)/} ``` 2. `parameterize` would strip all Unicode characters, leaving a blank string. Rails would report something like: ActionView::Template::Error (No route matches {:action=>"show", :controller=>"groups/milestones", :group_id=>#<Group id: 48, name: "ops-dev", path: "ops-dev", owner_id: nil, created_at: "2015-11-15 08:55:30", updated_at: "2015-12-02 06:23:26", type: "Group", description: "", avatar: "sha1.c71e73d51af1865c1bbbf6208e10044d46c9bb93.png", public: false>, :id=>"", :title=>"肯定不是中文的问题"} missing required keys: [:id]): This change uses the babosa library to create a better slug, which surprisingly isn't actually used by the global milestone controllers. Instead, they use the title passed as a query string for some reason. Closes https://github.com/gitlabhq/gitlabhq/issues/9881 Fix constraints
| * | | | | | Merge branch 'issue_1156'Douwe Maan2015-12-071-25/+29
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| | * | | | | Run custom Git hooks when creating or deleting branches through the UI. #1156Rubén Dávila2015-12-031-25/+29
| | |/ / / /
* | | | | | Merge branch 'master' into merge-if-greenZeger-Jan van de Weg2015-12-0730-235/+463
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge branch 'link-refs' into 'master' Robert Speicher2015-12-047-43/+131
| |\ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Recognize issue/MR/snippet/commit links as references. Fixes #3744 and #3745 See merge request !1933