summaryrefslogtreecommitdiff
path: root/lib/api
Commit message (Collapse)AuthorAgeFilesLines
* Return the number of marked todosapi-delete-todosRobert Schilling2016-07-191-1/+1
|
* Revert "Merge branch '18193-developers-can-merge' into 'master' "Robert Speicher2016-07-131-1/+1
| | | | | This reverts commit 9ca633eb4c62231e4ddff5466c723cf8e2bdb25d, reversing changes made to fb229bbf7970ba908962b837b270adf56f14098f.
* Merge branch 'dropdown-load-fix' into 'master' Douwe Maan2016-07-132-1/+6
|\ | | | | | | | | | | | | | | | | | | Dropdown loading time preformance fix ## What does this MR do? Optimizes the performance of the dropdown load time by just sending the required data to load the dropdown instead of the full object This MR aims to fix #17474 See merge request !5113
| * deletes swn filetiagonbotelho2016-07-131-0/+0
| |
| * fixes test according to four-phase test patterntiagonbotelho2016-07-121-0/+0
| |
| * removes basicprojectwithaccess and replaces it with basicprojectdetailstiagonbotelho2016-07-123-9/+1
| |
| * changes the usage of simpleprojectdetails to already implemented ↵tiagonbotelho2016-07-122-17/+0
| | | | | | | | basicprojectsdetails and changes the url to a more reader friendly format
| * adds basic functionality to the new endpoint of the apitiagonbotelho2016-07-123-19/+17
| |
| * implements filter_paramstiagonbotelho2016-07-122-0/+19
| |
| * changes string to symbol in paramtiagonbotelho2016-07-121-1/+1
| |
| * adds test to check json fields on simple request and changes the url request ↵tiagonbotelho2016-07-121-1/+1
| | | | | | | | format
| * fixes missing field on basicprojectdetailstiagonbotelho2016-07-121-0/+1
| |
| * changes the usage of simpleprojectdetails to already implemented ↵tiagonbotelho2016-07-122-19/+6
| | | | | | | | basicprojectsdetails and changes the url to a more reader friendly format
| * adds basic functionality to the new endpoint of the apitiagonbotelho2016-07-123-21/+25
| |
| * implements filter_paramstiagonbotelho2016-07-123-0/+21
| |
* | Refactor `Gitlab::GitAccess`Timothy Andrew2016-07-131-1/+1
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Don't use case statements for dispatch anymore. This leads to a lot of duplication, and makes the logic harder to follow. 2. Remove duplicated logic. - For example, the `can_push_to_branch?` exists, but we also have a different way of checking the same condition within `change_access_check`. - This kind of duplication is removed, and the `can_push_to_branch?` method is used in both places. 3. Move checks returning true/false to `UserAccess`. - All public methods in `GitAccess` now return an instance of `GitAccessStatus`. Previously, some methods would return true/false as well, which was confusing. - It makes sense for these kinds of checks to be at the level of a user, so the `UserAccess` class was repurposed for this. The prior `UserAccess.allowed?` classmethod is converted into an instance method. - All external uses of these checks have been migrated to use the `UserAccess` class 4. Move the "change_access_check" into a separate class. - Create the `GitAccess::ChangeAccessCheck` class to run these checks, which are quite substantial. - `ChangeAccessCheck` returns an instance of `GitAccessStatus` as well. 5. Break out the boolean logic in `ChangeAccessCheck` into `if/else` chains - this seems more readable. 6. I can understand that this might look like overkill for !4892, but I think this is a good opportunity to clean it up. - http://martinfowler.com/bliki/OpportunisticRefactoring.html
* Merge branch 'remove-branch-api' into 'master' Rémy Coutable2016-07-121-0/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api: expose {should,force}_remove_source_branch ## What does this MR do? Exposes the `should_remove_source_branch` and `force_remove_source_branch` booleans via the API. ## Are there points in the code the reviewer needs to double check? I don't think so. ## Why was this MR needed? See the commit message. ## What are the relevant issue numbers? N/A ## 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 - [ ] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [ ] 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 !5184
| * api: expose {should,force}_remove_source_branchBen Boeckel2016-07-111-0/+2
| | | | | | | | | | Workflows which use a bot to merge should remove branches if requested. Expose the flag so that bots can request know this.
* | entities: remove :description from MergeRequestBen Boeckel2016-07-111-1/+0
|/ | | | It is already part of ProjectEntity.
* API: Expose shared projects in a groupapi-shared-projectsRobert Schilling2016-07-081-0/+1
|
* Expose shared groups for projectsapi-shared-groupsRobert Schilling2016-07-081-0/+11
|
* Extract parts of LegacyDiffNote into DiffOnNote concern and move part of ↵Douwe Maan2016-07-061-3/+3
| | | | responsibility to other classes
* Rename MergeRequest methods that return commits or shas to be more clear and ↵Douwe Maan2016-07-061-2/+2
| | | | consistent
* Only allow Git Access on the allowed protocolPatricio Cano2016-07-051-2/+5
|
* Enable Style/EmptyLines cop, remove redundant onesrubocop/enable-cops-for-empty-linesGrzegorz Bizon2016-07-018-10/+0
|
* Merge branch 'todos-api' into 'master' Rémy Coutable2016-07-013-0/+108
|\ | | | | | | | | | | | | | | | | | | | | | | | | Add Todos API * Closes #14068 * Closes #14675 - [x] Implementation - [x] Tests - [x] Documentation - [x] Changelog See merge request !3188
| * Support creating a todo on issuables via APItodos-apiRobert Schilling2016-07-011-0/+30
| |
| * Expose target, filter by state as stringRobert Schilling2016-07-012-7/+7
| |
| * Add todos API documentation and changelogRobert Schilling2016-07-011-3/+1
| |
| * pass paginated array when deleting notesRobert Schilling2016-07-011-1/+1
| |
| * Expose action_nameRobert Schilling2016-07-011-1/+1
| |
| * Add user to project to see todosRobert Schilling2016-07-011-1/+1
| |
| * Move to helper, no instance variablesRobert Schilling2016-07-011-9/+9
| |
| * Fix rebaseRobert Schilling2016-07-011-4/+0
| |
| * Add Todos APIDouglas Barbosa Alexandre2016-07-013-0/+84
| |
* | Merge branch 'refactor-tags-branches-api' into 'master' Rémy Coutable2016-07-012-7/+7
|\ \ | |/ |/| | | | | | | | | | | | | Ensure that branch and tag names are given in API ## What does this MR do? It ensures that the tag or branch name is given. @rymai We talked in an earlier MR about this. See merge request !5012
| * Ensure that branch and tag names are given in APIrefactor-tags-branches-apiRobert Schilling2016-06-302-7/+7
| |
* | Add Application Setting to configure default Repository Path for new projectsshards-configAlejandro Rodríguez2016-06-291-0/+1
| |
* | Refactor repository paths handling to allow multiple git mount pointsAlejandro Rodríguez2016-06-291-11/+25
|/
* Enable Style/SpaceAfterComma Rubocop coprubocop/enable-space-after-copsGrzegorz Bizon2016-06-291-1/+1
|
* Remove redundant check in commit builds API endpointfix/builds-api-builds-for-commit-searchGrzegorz Bizon2016-06-281-2/+0
|
* Improve commit builds API endpoint RESTful behaviorGrzegorz Bizon2016-06-281-1/+3
| | | | | | | | 1. Return 404 if commit is not found (RESTful resource not found) 2. Return an empty array if pipeline is not found (resource present, no associated builds found) 3. Return an empty array if pipeline found but no builds there (resource present, no associated builds)
* Find all builds for commit if multiple pipelinesGrzegorz Bizon2016-06-281-3/+3
|
* Rename Licenses API to License Templates APIDrew Blessing2016-06-272-3/+3
|
* gitlab-org/gitlab-ce#17818 - add api call for issues by groupMarc Siegfriedt2016-06-231-0/+35
| | | | | rely only on IssuesFinder docs and changelog
* Fix builds API response not including commit dataGrzegorz Bizon2016-06-211-5/+1
|
* Merge branch '17521-gitlab-ci-yml-templates' into 'master' Jacob Schatz2016-06-214-32/+39
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GitLab CI Yaml template dropdown ## What does this MR do? Make it possible to select a dropdown for an easy start with GitLab CI. ## What are the relevant issue numbers? Closes #17521 ## TODO - [ ] Backend - [x] CHANGELOG item - [x] Fix rubocop failure - [x] API Support - [x] New tests - [x] Add disclaimer to the top of the gitlab-ci.yml - [ ] Frontend - [x] New tests See merge request !4411
| * fixup! override content methodZ.J. van de Weg2016-06-201-5/+1
| |
| * Incorporate reviewZ.J. van de Weg2016-06-201-1/+5
| |
| * Implement backend gitlab ci dropdownZ.J. van de Weg2016-06-201-1/+2
| | | | | | | | | | | | | | This commit builds on the groundwork in ee008e300b1ec0abcc90e6a30816ec0754cea0dd, which refactored the backend so the same code could be used for new dropdowns. In this commit its used for templates for the `.gitlab-ci.yml` files.