summaryrefslogtreecommitdiff
path: root/spec/requests/api
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch '22065-group-members-api-returning-null-user-details' into ↵Robert Speicher2016-09-201-9/+18
|\ | | | | | | | | | | | | | | | | | | | | | | 'master' Ensure invitees are not returned in Members API ## What are the relevant issue numbers? Closes #22065 See merge request !6370
| * Better assertion in API members specs22065-group-members-api-returning-null-user-detailsRémy Coutable2016-09-161-1/+3
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * Ensure invitees are not returned in Members APIRémy Coutable2016-09-161-9/+16
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'issue_20978' into 'master' Rémy Coutable2016-09-192-5/+24
|\ \ | | | | | | | | | | | | | | | | | | Allow to set request_access_enabled for groups and projects using API Closes #20978 See merge request !6359
| * | Allow to set request_access_enabled for groups and projects using APIFelipe Artur2016-09-192-5/+24
| | |
* | | Ensure validation messages are shown within the milestone formMark Fletcher2016-09-191-2/+21
| | | | | | | | | | | | | | | | | | | | | * Remove call to Milestone#save! instead just Milestone#save * Add safety specs to stop a regression Fixes #22033
* | | Add specs that target status failure22286-fix-missing-statusKamil Trzcinski2016-09-181-1/+1
|/ /
* | Fix API notes endpoint when posting only emojiZ.J. van de Weg2016-09-161-0/+9
|/
* Merge branch 'fix-commit-status' into 'master' Rémy Coutable2016-09-131-10/+29
|\ | | | | | | | | | | | | | | | | | | | | | | | | Fix an error where we were unable to create a CommitStatus for running state Due to severe refactoring of Pipeline we introduced regression in how CommitStatus is handled. We received an report that it's impossible to create a CommitStatus with state `running` when there were not previous status. The support for Commit Statuses should be simplified. Right now I'm doing minimal change to move forward and fix a bug, but I'll create a new MR that will move all logic that is now part of `lib/api/commit_statuses.rb` to separate service to simplify the implementation. This error happens due to the fact that we introduced additional status of builds: `created`. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/21345 See merge request !6107
| * Fix an error where we were unable to create a CommitStatus for running statefix-commit-statusKamil Trzcinski2016-09-121-10/+29
| |
* | Ensure specs on sorting of issues in API are deterministic on MySQL22057-fix-api-ossues-sorting-flaky-specsPaco Guzman2016-09-121-0/+3
|/ | | | MySQL could not have support for millisecond precision, depends on the MySQL version so we just create issues in different seconds in a deterministic way
* Merge branch 'fix-api-sorting' into 'master' Douwe Maan2016-09-121-6/+120
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix API issues sorting ## What does this MR do? Fix the sorting of issues in the API. ## Are there points in the code the reviewer needs to double check? Instead of removing the '_at' suffix manually, we could add those versions to the `Sortable` concern instead. ## Why was this MR needed? There were a couple of bugs: * The global and project-specific issues endpoints wouldn't sort at all. * Group sorting would work, but only if you applied two undocumented workarounds: * Always pass both `order_by` and `sort` (both are optional, so only one should be needed to change ordering). * Instead of passing `created_at` or `updated_at`, you needed to pass `created` or `updated`. This makes the API implementation match the docs. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] API support added - Tests - [x] Added for this feature/bug - [ ] 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 you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/983. See merge request !6281
| * Fix API issues sortingfix-api-sortingSean McGivern2016-09-091-6/+120
| |
* | Add notification_settings API callsMarc Siegfriedt2016-09-091-0/+89
|/ | | | Use NotificationSetting::EMAIL_EVENTS for params
* Code refactorKatarzyna Kobierska2016-09-071-4/+4
|
* Improve codeKatarzyna Kobierska2016-09-071-3/+3
|
* Code refactoringKatarzyna Kobierska2016-09-071-15/+16
|
* Fix errors and grammarKatarzyna Kobierska2016-09-071-8/+9
|
* Add tests for yaml content with errorsKatarzyna Kobierska2016-09-071-11/+17
|
* Move lint to api from ci/apiKatarzyna Kobierska2016-09-071-0/+41
|
* Higher code quality for fork to namespace APIOlaf Tomalka2016-09-061-4/+4
|
* Improved code quality on API fork namespace featureOlaf Tomalka2016-09-061-8/+22
|
* Added fork API paremeter for a specific namespaceOlaf Tomalka2016-09-061-1/+53
| | | | | | | | | Browser interface allows forking to an owned grup. This commit brings API up to speed by providing optional namespace parameter to fork API. This allows forking to users and groups under forker's control using their id or unique name. Fixes #21591
* Merge branch 'internal_recovery_api' into 'master' Douwe Maan2016-09-021-2/+2
|\ | | | | | | | | | | | | Minor edits to two_factor_recovery_codes API error catching Minor fixes to the `two_factor_recovery_codes` internal API making the conditionals more sane. Thanks @DouweM for pointing it out. See merge request !6000
| * Minor edits to two_factor_recovery_codes API error catchingDrew Blessing2016-08-311-2/+2
| |
* | Merge branch 'issue_19734' into 'master' Douwe Maan2016-09-011-1/+8
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Project tools visibility level ## part of #19734 ![project_features_access_level](/uploads/81ec7185d4e61d7578652020209af925/project_features_access_level.png) ## 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 !5606
| * | Project tools visibility levelFelipe Artur2016-09-011-1/+8
| | |
* | | Merge branch 'merge-request-sha-info' into 'master' Douwe Maan2016-09-011-1/+8
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge request sha info ## What does this MR do? Exposes `sha` and `merge_commit_sha` items in `MergeRequest` API endpoint data. ## Are there points in the code the reviewer needs to double check? No. ## Why was this MR needed? It's useful information. ## What are the relevant issue numbers? Closes #20456. ## Screenshots (if relevant) N/A ## 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 !5966
| * | entities: expose {,merge_commit_}sha in MergeRequestBen Boeckel2016-08-311-1/+8
| | | | | | | | | | | | Fixes #20456.
* | | Add BroadcastMessage API implementationRobert Speicher2016-08-311-0/+180
| | |
* | | Merge branch 'api-confidential-issues' into 'master' Robert Speicher2016-08-311-1/+64
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | API: Expose issue confidentiality flag. ## What does this MR do? This MR adds support for confidential issues the the API. The property as exposed and users can create or update confidential issues. ## What are the relevant issue numbers? Closes #17199 See merge request !5384
| * | | API: Expose issue#confidentialapi-confidential-issuesRobert Schilling2016-08-311-1/+64
| |/ /
* | | Don't expose a user's private token in the `/api/v3/user` API.Timothy Andrew2016-08-311-0/+1
|/ / | | | | | | | | | | - This would allow anyone with a personal access token (even a read-only token, once scopes are implemented) to escalate their access by obtaining the private token.
* | Merge branch 'prevent_authored_awardable_votes' into 'master' Robert Speicher2016-08-301-1/+17
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | prevent authored awardable thumbs votes ## What does this MR do? This MR should prevent users from upvoting or downvoting issues/merge requests/notes authored by them. ## What are the relevant issue numbers? Closes #20913 See merge request !5841
| * \ Merge branch 'master' into prevent_authored_awardable_votesbarthc2016-08-302-7/+17
| |\ \
| * | | prevent authored awardable thumbs votesbarthc2016-08-281-1/+17
| | |/ | |/| | | | | | | | | | | | | prevent authored awardable thumbs votes prevent authored awardable thumbs votes
* | | Merge branch ↵Douwe Maan2016-08-301-0/+25
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '19721-issues-created-through-api-do-not-notify-label-subscribers' into 'master' user is now notified when creating an issue through the api Previously when a issue was created through our API it would not mail label subscribers, this MR aims to fix that - [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) Closes #19721 See merge request !5720
| * | if issue is not valid we revert back to the old labels when updatingtiagonbotelho2016-08-301-1/+1
| | |
| * | refactors update issue api request and some minor commentstiagonbotelho2016-08-301-5/+17
| | |
| * | adds test to check whether or not an email is sent to label subscribers ↵tiagonbotelho2016-08-301-0/+13
| |/ | | | | | | after creating a new issue through the api
* | Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2016-08-269-2/+177
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | artifacts-from-ref-and-build-name * upstream/master: (107 commits) Fix external issue tracker "Issues" link leading to 404s Fix CHANGELOG entries related to 8.11 release. Fix changelog Reduce contributions calendar data payload Add lock_version to merge_requests table Add hover color to emoji icon Fix wrong Koding link Capitalize mentioned issue timeline notes Fix groups sort dropdown alignment Use icon helper Fix inline emoji text alignment Adds response mime type to transaction metric action when it's not HTML Moved two 8.11 changelog entries to 8.12 Fix markdown link in doc_styleguide.md Display project icon from default branch Reduce number of database queries on builds tab Update CHANGELOG Update Issue board documentation Label list shows all issues (opened or closed) with that label Update CHANGELOG ...
| * Merge branch 'capitalize-mentioned-issue-notes' into 'master' Douwe Maan2016-08-261-1/+1
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Capitalize mentioned issue timeline notes ## What does this MR do? Renames `mentioned in` to `Mentioned in` for issue timeline notes ## Are there points in the code the reviewer needs to double check? Shouldn't be any :smile: ## Why was this MR needed? Resolves existing UI inconsistency as all the other issue timeline notes have their first letter capitalized. ## What are the relevant issue numbers? Closes #21416 ## Screenshots (if relevant) Before: ![Screen_Shot_2016-08-25_at_10.52.47_AM](/uploads/61de0a7d65fd739a6e565f6e63d2f52d/Screen_Shot_2016-08-25_at_10.52.47_AM.png) After: ![Screen_Shot_2016-08-25_at_10.52.05_AM](/uploads/f67f353c285453306d6b80578b11e587/Screen_Shot_2016-08-25_at_10.52.05_AM.png) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [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 !6028
| | * Capitalize mentioned issue timeline notesClement Ho2016-08-251-1/+1
| | |
| * | Merge branch 'dz-merge-request-version'Douwe Maan2016-08-251-0/+49
| |\ \ | | |/ | |/|
| | * Merge branch 'master' into dz-merge-request-versionDmitriy Zaporozhets2016-08-221-2/+4
| | |\ | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * \ Merge branch 'master' into dz-merge-request-versionDmitriy Zaporozhets2016-08-1910-38/+303
| | |\ \ | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * \ \ Merge branch 'master' into dz-merge-request-versionDmitriy Zaporozhets2016-08-1537-1162/+1450
| | |\ \ \ | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * | | | Add single merge request diff API endpointDmitriy Zaporozhets2016-08-121-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| | * | | | Add API to list merge request diff versionsDmitriy Zaporozhets2016-08-031-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Merge branch 'internal_recovery_api' into 'master' Robert Speicher2016-08-241-0/+62
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add internal API to recovery 2FA ## What does this MR do? Add an internal API to make SSH 2FA recovery possible. Related to gitlab-org/gitlab-shell!74 See merge request !5510