Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge remote-tracking branch 'origin/master' into dockerfile-templatesdockerfile-templates | Kamil Trzcinski | 2016-12-18 | 2868 | -35378/+79204 |
|\ | |||||
| * | Merge branch '25741_enable_multiline_operation_indentation_rubocop_rule' ↵ | Sean McGivern | 2016-12-17 | 21 | -44/+45 |
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Enable Style/MultilineOperationIndentation in Rubocop Fixes #25741 See merge request !8125 | ||||
| | * | Enable Style/MultilineOperationIndentation in Rubocop, fixes #25741 | Rydkin Maxim | 2016-12-16 | 21 | -44/+45 |
| | | | |||||
| * | | Merge branch 'zj-mattermost-session' into 'master' | Grzegorz Bizon | 2016-12-17 | 4 | -0/+227 |
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mattermost session This branch is based of tag v8.14.4 so I could test on an instance better. But stuck on the tests, as setting up the whole doorkeeper stuff doesn't seem to be supported for testing. See merge request !8033 | ||||
| | * | | Fix rubocop failureszj-mattermost-session | Kamil Trzcinski | 2016-12-17 | 2 | -17/+23 |
| | | | | |||||
| | * | | Store mattermost_url in settings | Kamil Trzcinski | 2016-12-17 | 4 | -13/+35 |
| | | | | |||||
| | * | | Improve Mattermost Session specs | Kamil Trzcinski | 2016-12-17 | 2 | -26/+50 |
| | | | | |||||
| | * | | Ensure the session is destroyed | Z.J. van de Weg | 2016-12-17 | 1 | -3/+5 |
| | | | | |||||
| | * | | Improve session tests | Z.J. van de Weg | 2016-12-17 | 3 | -45/+74 |
| | | | | |||||
| | * | | Setup mattermost session | Z.J. van de Weg | 2016-12-17 | 2 | -0/+144 |
| |/ / | |||||
| * | | Merge branch 'cleaner-merge-commit-messages' into 'master' | Sean McGivern | 2016-12-17 | 9 | -21/+242 |
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | Cleaner merge commit messages Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/23462 See merge request !7722 | ||||
| | * | | Change closes issues reference text on MergeRequest#merge_commit_message to ↵ | Gabriel Gizotti | 2016-12-16 | 3 | -6/+5 |
| | | | | | | | | | | | | | | | | match existing text generated by the system | ||||
| | * | | add feature specs to test toggling of merge commit message between message ↵ | Gabriel Gizotti | 2016-12-16 | 2 | -3/+75 |
| | | | | | | | | | | | | | | | | with description and without | ||||
| | * | | add eslint disable prefix for prefer-arrow-callback rule on header of ↵ | Gabriel Gizotti | 2016-12-16 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | merge_request.js file | ||||
| | * | | Add feature spec to verify all 4 different states of closing issues message ↵ | Gabriel Gizotti | 2016-12-16 | 1 | -0/+57 |
| | | | | | | | | | | | | | | | | on Merge Request show page. | ||||
| | * | | add guard clause to MergeRequest#issues_mentioned_but_not_closing | Gabriel Gizotti | 2016-12-16 | 1 | -9/+6 |
| | | | | |||||
| | * | | add js prefix to classes used to toggle description on commit message in ↵ | Gabriel Gizotti | 2016-12-16 | 2 | -10/+10 |
| | | | | | | | | | | | | | | | | merge request | ||||
| | * | | Show either description or closes issues references on ↵ | Gabriel Gizotti | 2016-12-16 | 3 | -12/+12 |
| | | | | | | | | | | | | | | | | MergeRequest#merge_commit_message so closes issues references are not duplicated | ||||
| | * | | Unify commit message listeners in one function | Gabriel Gizotti | 2016-12-16 | 1 | -11/+6 |
| | | | | |||||
| | * | | better text for mentioned but not closed | Gabriel Gizotti | 2016-12-16 | 1 | -1/+1 |
| | | | | |||||
| | * | | add mentioned but not closed message to the same line as closes issueswq | Gabriel Gizotti | 2016-12-16 | 1 | -13/+14 |
| | | | | |||||
| | * | | extract duplicate logic into a variable on _commit_message_container | Gabriel Gizotti | 2016-12-16 | 1 | -2/+3 |
| | | | | |||||
| | * | | Better `Closes issues` text for MergeRequest#merge_commit_message | Gabriel Gizotti | 2016-12-16 | 1 | -1/+2 |
| | | | | |||||
| | * | | move javascript code from _commit_message_container view to ↵ | Gabriel Gizotti | 2016-12-16 | 2 | -21/+26 |
| | | | | | | | | | | | | | | | | javascripts/merge_request.js | ||||
| | * | | backend completely drives creation of merge commit message | Gabriel Gizotti | 2016-12-16 | 5 | -26/+37 |
| | | | | |||||
| | * | | Remove unnecessary code from MergeRequest#issues_mentioned_but_not_closing | Gabriel Gizotti | 2016-12-16 | 1 | -4/+2 |
| | | | | |||||
| | * | | describe #closes_issues and describe # #issues_mentioned_but_not_closing on ↵ | Gabriel Gizotti | 2016-12-16 | 1 | -24/+14 |
| | | | | | | | | | | | | | | | | merge_request_spec.rb | ||||
| | * | | only render MR description toggle javascript if description is available | Gabriel Gizotti | 2016-12-16 | 1 | -20/+19 |
| | | | | |||||
| | * | | keep branch being merged, MR title and MR reference in merge commit message ↵ | Gabriel Gizotti | 2016-12-16 | 1 | -2/+8 |
| | | | | | | | | | | | | | | | | when using description | ||||
| | * | | Add toggle links for using default message and description on change merge ↵ | Gabriel Gizotti | 2016-12-16 | 2 | -0/+29 |
| | | | | | | | | | | | | | | | | commit message container | ||||
| | * | | shorter lines on MergeRequest#merge_commit_message | Gabriel Gizotti | 2016-12-16 | 1 | -2/+9 |
| | | | | |||||
| | * | | only look for issues mentioned on description on ↵ | Gabriel Gizotti | 2016-12-16 | 2 | -3/+5 |
| | | | | | | | | | | | | | | | | MergeRequest#issues_mentioned_but_not_closing | ||||
| | * | | better mentioned but not closing message and icon | Gabriel Gizotti | 2016-12-16 | 2 | -9/+2 |
| | | | | |||||
| | * | | diplays mentioned but not merged message on MR show page | Gabriel Gizotti | 2016-12-16 | 2 | -0/+20 |
| | | | | |||||
| | * | | introduce MergeRequest#issues_mentioned_but_not_closing | Gabriel Gizotti | 2016-12-16 | 2 | -15/+46 |
| | | | | |||||
| | * | | Remove whole description from #merge_commit_message and add add closed issues | Gabriel Gizotti | 2016-12-16 | 2 | -5/+12 |
| | | | | |||||
| * | | | Merge branch '23573-sort-functionality-for-project-member' into 'master' | Douglas Barbosa Alexandre | 2016-12-17 | 13 | -13/+356 |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add sorting functionality for group/project members ## What does this MR do? This MR adds sort functionality to group/project members list. ## Are there points in the code the reviewer needs to double check? No. ## Why was this MR needed? This MR helps the user to manage and find users of a group/project fast. It also improves UX. ## Screenshots (if relevant) **Before** ![before-pr-member](/uploads/412bfdfddc63e48a62e2095d664660b5/before-pr-member.png) **After** ![pr-after](/uploads/ae23a5c7689b46d26a948f22a0ac2360/pr-after.png) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - Tests - [x] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #21622 Closes #23573 See merge request !7032 | ||||
| | * | | | Fix query in Projects::ProjectMembersController to fetch members23573-sort-functionality-for-project-member | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -2/+2 |
| | | | | | |||||
| | * | | | Improve test for sort dropdown on members page | Nur Rony | 2016-12-16 | 3 | -1/+19 |
| | | | | | |||||
| | * | | | Fix sort dropdown alignment | Nur Rony | 2016-12-16 | 1 | -2/+2 |
| | | | | | |||||
| | * | | | Undo changes on members search button stylesheet | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -16/+15 |
| | | | | | |||||
| | * | | | Use factories to create project/group membership on specs | Douglas Barbosa Alexandre | 2016-12-16 | 2 | -4/+4 |
| | | | | | |||||
| | * | | | Remove unused id from shared members sort dropdown | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -1/+1 |
| | | | | | |||||
| | * | | | Fix sort functionality on project/group members to return invited users | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -4/+15 |
| | | | | | |||||
| | * | | | Refactor MembersHelper#filter_group_project_member_path | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -5/+1 |
| | | | | | |||||
| | * | | | Remove unnecessary curly braces from sort dropdown partial | Douglas Barbosa Alexandre | 2016-12-16 | 1 | -1/+1 |
| | | | | | |||||
| | * | | | Sort group/project members alphabetically by default | Douglas Barbosa Alexandre | 2016-12-16 | 4 | -3/+17 |
| | | | | | |||||
| | * | | | Add feature spec for sort functionality on group/project members list | Douglas Barbosa Alexandre | 2016-12-16 | 2 | -0/+164 |
| | | | | | |||||
| | * | | | Add option to sort group/project members by access level | Douglas Barbosa Alexandre | 2016-12-16 | 2 | -0/+20 |
| | | | | | |||||
| | * | | | Fix sort functionality for group/project members | Douglas Barbosa Alexandre | 2016-12-16 | 3 | -12/+28 |
| | | | | |