When updating a merge request via the `/merge` endpoint we
check the `mergeable` and `mergeable_state` status, these will return
`false` if the application option only_allow_merge_if_pipeline_succeeds is
enabled. We should skip CI checks if the request uses the
merge_when_pipeline_succeeds param

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/22740

Removed all the unnecessary include of `WaitForAjax` and `ApiHelpers` in the specs.
Removed unnecessary usage of `api:true`

All the other endpoints of `/project/:id/merge_request` work with
`:merge_request_iid`, except for `approvals`, `approve`, and
`unapprove`. This commit fixes that.

Closes gitlab-org/gitlab-ee#1875

Comments for a merge request should be obtained to the `notes` endpoint.

Comments for a merge request should be obtained to the `notes` endpoint.

- As opposed to the `id` that was previously being used.
- This brings the API routes closer to the web interface's routes.
- This is specific to API v4.

Specs that are EE-only need to be updated to use IIDs instead of IDs

- As opposed to the `id` that was previously being used.
- This brings the API routes closer to the web interface's routes.
- This is specific to API v4.

 Rename column in the database
 Rename fields related to import/export feature
 Rename API endpoints
 Rename documentation links
 Rename the rest of occurrences in the code
 Replace the images that contain the words "build succeeds" and docs referencing to them
 Make sure pipeline is green and nothing is missing.

# Conflicts:
#	app/controllers/projects/merge_requests_controller.rb
#	app/serializers/merge_request_entity.rb
#	app/services/merge_requests/merge_when_pipeline_succeeds_service.rb
#	app/services/merge_requests/refresh_service.rb
#	app/views/projects/merge_requests/merge.js.haml
#	app/views/projects/merge_requests/widget/open/_accept.html.haml
#	doc/api/merge_requests.md
#	doc/api/projects.md
#	lib/api/entities.rb
#	lib/api/v3/entities.rb
#	lib/api/v3/merge_requests.rb
#	spec/services/merge_requests/merge_when_pipeline_succeeds_service_spec.rb
#	spec/services/merge_requests/refresh_service_spec.rb

 Rename column in the database
 Rename fields related to import/export feature
 Rename API endpoints
 Rename documentation links
 Rename the rest of occurrences in the code
 Replace the images that contain the words "build succeeds" and docs referencing to them
 Make sure pipeline is green and nothing is missing.

updated doc images

renamed only_allow_merge_if_build_succeeds in projects and fixed references

more updates

fix some spec failures

fix rubocop offences

fix v3 api spec

fix MR  specs

fixed issues with partials

fix MR spec

fix alignment

add missing v3 to v4 doc

wip - refactor v3 endpoints

fix specs

fix a few typos

fix project specs

copy entities fully to V3

fix  entity error

more fixes

fix failing specs

fixed missing entities in V3 API

remove comment

updated code based on feedback

typo

fix spec

This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b.

This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b.

Ensure that only privileged users can access merge requests in the API

See merge request !2053

New endpoints are:

POST :project_id/(issues|merge_requests)/(:issue_id|:merge_request_id)/time_estimate"

POST :project_id/(issues|merge_requests)/(:issue_id|:merge_request_id)/reset_time_estimate"

POST :project_id/(issues|merge_requests)/(:issue_id|:merge_request_id)/add_spent_time"

POST :project_id/(issues|merge_requests)/(:issue_id|:merge_request_id)/reset_spent_time"

GET  :project_id/(issues|merge_requests)/(:issue_id|:merge_request_id)/time_stats"