summaryrefslogtreecommitdiff
path: root/app/controllers/projects/merge_requests_controller.rb
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'fix/rename-mwbs-to-merge-when-pipeline-succeeds' into 'master' Kamil Trzciński2016-12-051-4/+10
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename MWBS service to Merge When Pipeline Succeeds ## What does this MR do? This MR renames Merge When Build Succeeds feature to Merge When Pipeline Succeeds, because we recently changed the behavior, see https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6675. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [x] All builds are passing ## What are the relevant issue numbers? Closes #23354 See merge request !7135
| * Merge branch 'master' into fix/rename-mwbs-to-merge-when-pipeline-succeedsGrzegorz Bizon2016-11-301-8/+10
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (110 commits) Rewrite an HTTP link to use HTTPS Edit /spec/features/profiles/preferences_spec.rb to match changes in 084d90ac Add blue back to sub nav active Remove JSX/React eslint plugins. Fix a transient spec failure Adds hoverstates for collapsed Issue/Merge Request sidebar Moved groups above projects Add StackProf to the Gemfile, along with a utility to get a profile for a spec Update Sidekiq-cron to fix compatibility issues with Sidekiq 4.2.1 Add a CHANGELOG entry Alert user when logged in user email is not the same as the invitation Expose timestamp in build entity used by serializer Rename `MergeRequest#pipeline` to `head_pipeline` Remove unnecessary database indexes CE-specific changes gitlab-org/gitlab-ee#1137 Fixing typo & Clarifying Key name fix started_at check fix blob controller spec failure - updated not to use file-path- fix blob controller spec failure Merge branch 'jej-use-issuable-finder-instead-of-access-check' into 'security' ... Conflicts: app/controllers/projects/merge_requests_controller.rb lib/api/merge_requests.rb spec/requests/api/merge_requests_spec.rb
| * \ Merge branch 'master' into fix/rename-mwbs-to-merge-when-pipeline-succeedsGrzegorz Bizon2016-11-251-9/+6
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (312 commits) Fix bad selection on dropdown menu for tags filter Fixed issue boards scrolling with a lot of lists & issues You can only assign default_branch when editing a project ... Don't convert data which already is the target type Stop supporting Google and Azure as backup strategies renames some of the specs and adds changelog entry Fixed dragging issue moving wrong issue after multiple drags of issue Fixed issue boards issue sorting when dragging issue into list Rephrase some system notes to be compatible with new system note style Add missing JIRA file that redirects to the new location Fix documentation to create the `pg_trm` extension before creating the DB Document that we always use `do...end` for `before` in RSpec Backport Note#commands_changes from EE Log mv_namespace parameters Add default_branch attr to Project API payload in docs. Fix title case to sentence case properly escape username validation error message flash Remove header ids from University docs Add missing documentation. Added test that checks the correct select box is there for the LFS ... ... Conflicts: app/services/system_note_service.rb spec/features/merge_requests/merge_when_pipeline_succeeds_spec.rb spec/services/merge_requests/merge_when_pipeline_succeeds_service_spec.rb spec/services/system_note_service_spec.rb
| * | | Rename MWBS service to Merge When Pipeline SucceedsGrzegorz Bizon2016-11-211-4/+9
| | | |
* | | | Replace references to MergeRequestDiff#commits with st_commitsuse-st-commits-where-possibleAdam Niedzielski2016-12-011-2/+2
| |_|/ |/| | | | | | | | | | | | | | when we care only about the number of commits We do not have to instantiate all objects in this case.
* | | Rename `MergeRequest#pipeline` to `head_pipeline`fix/rename-merge-request-head-pipelineGrzegorz Bizon2016-11-291-7/+8
| | |
* | | Prevent error when submitting a merge request and pipeline is not defined24860-actionview-template-error-undefined-method-size-for-nil-nilclassAdam Niedzielski2016-11-281-1/+1
| |/ |/|
* | Do not use MergeRequest#commits in define_pipelines_varshot-fix-define-pipeline-varsAdam Niedzielski2016-11-221-5/+2
| | | | | | | | | | MergeRequest#commits requires merge_request_diff to be present. This is not the case when creating a new merge request.
* | Merge branch 'hide-empty-merge-request-diffs' into 'master' Sean McGivern2016-11-211-4/+4
|\ \ | | | | | | | | | | | | Fix errors happening when source branch of merge request is removed and then restored See merge request !7568
| * | Hide box with CI status when there are no changes in merge requestAdam Niedzielski2016-11-211-1/+1
| | |
| * | Return valid response in Projects::MergeRequestsController#ci_status when ↵Adam Niedzielski2016-11-211-1/+1
| | | | | | | | | | | | diff_head_sha is nil
| * | Hide empty merge request diffs when comparing versionsAdam Niedzielski2016-11-211-2/+2
| |/
* | Merge branch 'ee-870-backport' into 'master' Sean McGivern2016-11-211-1/+1
|\ \ | |/ |/| | | | | Backport some changes done from Time Tracking feature in EE. See merge request !7604
| * Backport some changes done from Time Tracking feature in EE.ee-870-backportRuben Davila2016-11-181-1/+1
| |
* | More aggressively preload on merge request and issue index pagesDouwe Maan2016-11-181-1/+0
|/
* Ensure labels are loaded on "show" methods of MRAlex Sanford2016-11-111-0/+1
| | | | | Previously, they were only being loaded on the actual `show` action, but not the actions representing the tabs (commits, builds, etc).
* Merge branch 'show-status-from-branch' into 'master' Rémy Coutable2016-11-041-2/+12
|\ | | | | | | | | | | | | Show pipeline status from branch and commit than only commit Closes #23615 See merge request !7034
| * Merge remote-tracking branch 'upstream/master' into show-status-from-branchLin Jen-Shin2016-11-021-1/+1
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (126 commits) Allow to search for user by secondary email address in the admin interface Rename :name search parameter to :search_query at /admin/users Fix project features default values Add a link to the Issue Boards API in main README Fix Markdown styling inside reference links Fix relative links in Markdown wiki when displayed in "Project" tab Flexbox webkit prefixes Tests update Tests update Changed where merge request link is New todos blank state Implement CreateMembers service to make controller thin Do not show tooltip for active element (!7105) Update CHANGELOG Assign local_assigns[:subject] to a variable on the shared label partial Use select instead of pluck on Project.group_ids Skip authorization check when searching for labels on IssuableFinder Use label subject to calculate number of issues/mrs within the group Remove unnecessary includes(:priorities) on Projects::LabelsController Add 8.13.2 CHANGELOG entries ...
| * | Pass `@ref` along so we know which pipeline to showLin Jen-Shin2016-10-241-2/+12
| | | | | | | | | | | | Closes #23615
* | | Apply `*_params_ce + *_params_ee` pattern to MergeRequestsControllerRobert Speicher2016-11-041-6/+21
| |/ |/|
* | add "x of y tasks completed" on issuableGuilherme Salazar2016-10-281-1/+1
|/ | | | | | fix issues pointed out in !6527 add task completion status feature to CHANGELOG
* Fix two CI endpoints for MRs where the source project is deletedNick Thomas2016-10-201-1/+1
|
* Fix the merge request view when source projects or branches are removedNick Thomas2016-10-201-1/+1
|
* Use present? instead of presence on Projects::MergeRequestsControllerDouglas Barbosa Alexandre2016-10-191-1/+4
|
* Remove project_labels from Projects::ApplicationControllerDouglas Barbosa Alexandre2016-10-191-1/+0
|
* List group labels on project labels pageDouglas Barbosa Alexandre2016-10-191-1/+1
|
* Allow users to apply group labels on Issues/MRsDouglas Barbosa Alexandre2016-10-191-1/+4
|
* Merge branch '21444-pipeliens-new-mr' into 'master' Fatih Acet2016-10-191-6/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add pipelines tab to new MR #### What does this MR do? Adds pipelines tab to new MRs #### Screenshots (if relevant) ![Screen_Shot_2016-10-10_at_10.23.27_AM](/uploads/6c3f8f2be0cf9ba7cc78f6d918307ec0/Screen_Shot_2016-10-10_at_10.23.27_AM.png) ![Screen_Shot_2016-10-11_at_8.59.45_AM](/uploads/e67577d92327eafef6f04073f3d94212/Screen_Shot_2016-10-11_at_8.59.45_AM.png) #### What are the relevant issue numbers? Closes #21444 See merge request !6238
| * Extract pipeline vars in merge requests controllerGrzegorz Bizon2016-10-181-5/+13
| |
| * Use all pipelines variable when creating merge requestGrzegorz Bizon2016-10-181-2/+2
| |
| * Remove obsolete variable assigned to MR widget viewGrzegorz Bizon2016-10-181-1/+0
| |
* | Merge remote-tracking branch 'origin/master' into 22191-delete-dynamic-envs-mrKamil Trzcinski2016-10-181-5/+15
|\ \ | |/
| * Merge branch 'master' into merge-conflicts-editor-2Sean McGivern2016-10-171-1/+25
| |\
| * | Allow setting content for resolutionsSean McGivern2016-10-131-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When reading conflicts: 1. Add a `type` field. `text` works as before, and has `sections`; `text-editor` is a file with ambiguous conflict markers that can only be resolved in an editor. 2. Add a `content_path` field pointing to a JSON representation of the file's content for a single file. 3. Hitting `content_path` returns a similar datastructure to the `file`, but without the `content_path` and `sections` fields, and with a `content` field containing the full contents of the file (with conflict markers). When writing conflicts: 1. Instead of `sections` being at the top level, they are now in a `files` array. This matches the read format better. 2. The `files` array contains file hashes, each of which must contain: a. `new_path` b. `old_path` c. EITHER `sections` (which works as before) or `content` (with the full content of the resolved file).
* | | Fix remaining specs failuresKamil Trzcinski2016-10-181-1/+6
| | |
* | | Refactor code to use available and stopped statuses and refactor views to ↵Kamil Trzcinski2016-10-171-1/+1
| | | | | | | | | | | | use separate renders
* | | Add environment stop action [ci skip]Kamil Trzcinski2016-10-171-0/+1
| |/ |/|
* | Add test, fix merge errorzj-show-deployed-time-agoZ.J. van de Weg2016-10-141-1/+2
| |
* | Minor style improvementZ.J. van de Weg2016-10-141-14/+6
| |
* | Rename method in testZ.J. van de Weg2016-10-141-18/+23
| |
* | Moved ci_status environments logic to new action ci_envrionments_status and ↵Luke Bennett2016-10-141-18/+20
| | | | | | | | set up frontend polling
* | Refactor ci_status on MergeRequestControllerZ.J. van de Weg2016-10-141-25/+20
| |
* | Show what time ago a MR was deployedZ.J. van de Weg2016-10-141-1/+30
|/
* Merge branch 'assign-issues-for-merge-request-18876' into 'master' Douwe Maan2016-10-111-1/+22
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ability to bulk assign issues to author of merge request ## What does this MR do? Provides a link to auto-assign issues to the author of a merge request, when they are mentioned as being closed by the MR. ## Are there points in the code the reviewer needs to double check? ## Why was this MR needed? To help avoid working on a MR without having assigned related issues to self ## What are the relevant issue numbers? Fixes #18876 ## Screenshots (if relevant) ![ScreenShot-P216](/uploads/1af5e71a0a0ff0a60c5d7b54c0e09d9c/ScreenShot-P216.png) ## Tasks - [x] Refactor or move away from using `BulkUpdateService` - [x] ~~Consider alternate link message when only a subset of issues will be assigned~~ - [x] Minimize repeated calls to expensive `closes_issues` method - [x] Move away from using inflector for pluralization and fix flash message - [x] Change auth `before_action` and fallback to error flash message - [x] Shouldn't overwrite current assignee if one exists ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] ~~[Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)~~ - [x] ~~API support added~~ - Tests - [x] Added for this feature/bug - [x] All builds are passing - [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 you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5725
| * fixup! Added link to bulk assign issues to MR author. (Issue #18876)James Edwards-Jones2016-10-071-0/+2
| |
| * Added link to bulk assign issues to MR author. (Issue #18876)James2016-10-061-1/+20
| |
* | MergeRequest new form load diff asynchronouslynew-merge-requests-commit-tab-activePaco Guzman2016-10-061-23/+43
|/
* Add a /wip slash commandThomas Balthazar2016-10-031-1/+1
| | | | It toggles the 'WIP' prefix in the MR title.
* Change callbackKatarzyna Kobierska2016-09-301-1/+1
|
* Callback close_merge_request_without_source_projectKatarzyna Kobierska2016-09-301-7/+8
|