- 13 Dec, 2016 2 commits
-
-
Grzegorz Bizon authored
-
Filipa Lacerda authored
-
- 12 Dec, 2016 2 commits
-
-
victorwu authored
-
Kamil Trzcinski authored
-
- 10 Dec, 2016 2 commits
-
-
Mike Greiling authored
-
Mike Greiling authored
-
- 09 Dec, 2016 2 commits
-
-
Douwe Maan authored
Replace MR access checks with use of MergeRequestsFinder Split from !2024 to partially solve https://gitlab.com/gitlab-org/gitlab-ce/issues/23867
- Potentially untested - No test coverage - Test coverage of some sort exists (a test failed when error raised) - Test coverage of return value (a test failed when nil used) - Permissions check tested - [x] app/finders/notes_finder.rb:17 - [x] app/views/layouts/nav/_project.html.haml:80 [`.count`] - [x] app/controllers/concerns/creates_commit.rb:84 - [x] app/controllers/projects/commits_controller.rb:24 - [x] app/controllers/projects/compare_controller.rb:56 - [x] app/controllers/projects/discussions_controller.rb:29 - [x] app/controllers/projects/todos_controller.rb:27 - [x] app/models/commit.rb:268 - [x] lib/gitlab/search_results.rb:71 - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_267_266 Memoize ` merged_merge_request(current_user)` - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_248_247 Expected side effect for `merged_merge_request!`, consider `skip_authorization: true`. - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_269_269 Scary use of unchecked `merged_merge_request?` See merge request !2033 -
Robert Speicher authored
Don't accidentally mark unsafe diff lines as HTML safe Fixes potential XSS issue when a legacy diff note is created on a merge request whose diff contained HTML See https://gitlab.com/gitlab-org/gitlab-ce/issues/25249 See merge request !2040
-
- 08 Dec, 2016 2 commits
-
-
Rydkin Maxim authored
fix features_visibility_spec.rb remove strange part of spec fix conditions of comments tab disappearing and fix spec generate changelog rewrite spec for more coplex check step-by-step move conditional logic into helper and fix changelog fix indentation in helper
-
Dmitriy Zaporozhets authored
* add parent_id field to namespaces table to store relation with nested groups * create routes table to keep information about full path of every group and project * project/group lookup by full path from routes table Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 07 Dec, 2016 3 commits
-
-
Filipa Lacerda authored
-
Filipa Lacerda authored
Fix error
-
Filipa Lacerda authored
Updates changelog with MR ID
-
- 06 Dec, 2016 3 commits
-
-
Grzegorz Bizon authored
-
Grzegorz Bizon authored
-
Grzegorz Bizon authored
-
- 05 Dec, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 02 Dec, 2016 2 commits
-
-
Oswaldo Ferreira authored
-
mattl authored
-
- 01 Dec, 2016 3 commits
-
-
Guyzmo authored
Adding the necessary API for the new /snippets Restful resource added with this commit. Added a new Grape class `Snippets`, as well as a `PersonalSnippet` entity. Issue: #20042 Merge-Request: !6373 Signed-off-by: Guyzmo <guyzmo+gitlab+pub@m0g.net>
-
Sean McGivern authored
-
Sean McGivern authored
Instead of doing n queries for n states, do one query to get all the counts grouped by state, and figure out what the count is for each state is from that. We can still cache the individual counts (it can't hurt), but this will help with initial load. Note that the `opened` scope on `Issuable` includes the `opened` and `reopened` states, which is why there's a special case.
-
- 30 Nov, 2016 2 commits
-
-
Mike Greiling authored
-
Mike Greiling authored
-
- 28 Nov, 2016 4 commits
-
-
David Wagner authored
Make sort and filter dropdowns look the same and tweak their icon and colors according to #24150. Signed-off-by: David Wagner <david@marvid.fr>
-
Sean McGivern authored
-
Sean McGivern authored
In the browser, we remove the + and - signs from the front of a diff line because we add them in with CSS, so they aren't copied. We can't do that in an email, because the CSS isn't supported, so we should keep them in that case.
-
- 26 Nov, 2016 1 commit
-
-
Gabriel Mazetto authored
-
- 23 Nov, 2016 2 commits
-
-
Rémy Coutable authored
Also create a new WorkhorseRequest concern Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Valery Sizov authored
-
- 22 Nov, 2016 4 commits
-
-
blackst0ne authored
Added tests Added changelog entry Resolved all issues in code
-
Annabel Dunstone Gray authored
-
Grzegorz Bizon authored
-
Grzegorz Bizon authored
-
- 21 Nov, 2016 2 commits
-
-
Luke "Jared" Bennett authored
-
Luke "Jared" Bennett authored
Review changes
-
- 18 Nov, 2016 3 commits
-
-
Felipe Artur authored
-
Douwe Maan authored
-
Luke "Jared" Bennett authored
Added new spec step Added empty state to dashboard Split empty and filtered empty states Moved empty_state icons into their own folder and DRY up empty state html Fixed failing spec Added to groups page Review changes
-