Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Enable frozen string in lib/api and lib/backup | gfyoung | 2018-09-29 | 1 | -0/+2 |
| | | | | | | | | | | Partially addresses #47424. Had to make changes to spec files because stubbing methods on frozen objects is a mess in RSpec and leads to failures: https://github.com/rspec/rspec-mocks/issues/1190 | ||||
* | Disable existing offenses for the CodeReuse cops | Yorick Peterse | 2018-09-11 | 1 | -0/+2 |
| | | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop. | ||||
* | Merge request and commit discussions API | Jan Provaznik | 2018-05-01 | 1 | -17/+13 |
| | |||||
* | Add discussion APIjprovazn-api | Jan Provaznik | 2018-03-07 | 1 | -66/+28 |
| | | | | | * adds basic discussions API for issues and snippets * reorganizes notes specs (so same tests can be used for all noteable types - issues, MRs, snippets) | ||||
* | support ordering of project notes in notes api | haseeb | 2017-11-29 | 1 | -1/+6 |
| | |||||
* | add `#with_metadata` scope to remove a N+1 from the notes' API | micael.bergeron | 2017-11-21 | 1 | -2/+2 |
| | |||||
* | Create system notes for MR too, improve doc + clean up code | Jarka Kadlecova | 2017-09-14 | 1 | -7/+2 |
| | |||||
* | Support discussion lock in the API | Jarka Kadlecova | 2017-09-14 | 1 | -0/+7 |
| | |||||
* | API: Use defined project requirementsrs-api-use-project-requirements | Robert Schilling | 2017-08-31 | 1 | -1/+1 |
| | |||||
* | Conditionally destroy a ressource | Robert Schilling | 2017-08-28 | 1 | -3/+3 |
| | |||||
* | API: Respect the 'If-Unmodified-Since' for delete endpoints | Robert Schilling | 2017-08-28 | 1 | -0/+2 |
| | |||||
* | Re-enable SqlInjection and CommandInjection | Brian Neel | 2017-08-08 | 1 | -1/+1 |
| | |||||
* | Update grape gemdz-update-grape | Dmitriy Zaporozhets | 2017-07-20 | 1 | -0/+1 |
| | | | | | | | | New version of the gem returns 200 status code on delete with content instead of 204 so we explicitly set status code to keep existing behavior Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Enable Style/DotPosition Rubocop :cop: | Grzegorz Bizon | 2017-06-21 | 1 | -2/+2 |
| | |||||
* | Remove the User#is_admin? method | blackst0ne | 2017-04-09 | 1 | -1/+1 |
| | |||||
* | API: Make the /notes endpoint work with noteable iid instead of id | Toon Claes | 2017-03-27 | 1 | -5/+9 |
| | | | | | | | | | | | | | | | | In API V4 all endpoints were changed so Merge Requests and Issues should be referred by iid, instead of id. Except the /notes endpoint was forgotten. So change the endpoints from: - /projects/:id/issues/:issue_id/notes - /projects/:id/merge_requests/:merge_request_id/notes To: - /projects/:id/issues/:issue_iid/notes - /projects/:id/merge_requests/:merge_request_iid/notes For Project Snippets nothing changes. | ||||
* | Add `requirements: { id: %r{[^/]+} }` for all projects and groups namespaced ↵ | Rémy Coutable | 2017-03-16 | 1 | -1/+1 |
| | | | | | | API routes Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Return 204 for delete endpoints | Robert Schilling | 2017-02-28 | 1 | -2/+0 |
| | |||||
* | Revert "Prefer leading style for Style/DotPosition" | Douwe Maan | 2017-02-23 | 1 | -2/+2 |
| | | | | This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b. | ||||
* | Prefer leading style for Style/DotPosition | Douwe Maan | 2017-02-23 | 1 | -2/+2 |
| | |||||
* | Enable Style/ColonMethodCall | Douwe Maan | 2017-02-23 | 1 | -1/+1 |
| | |||||
* | Enable Style/MutableConstant | Douwe Maan | 2017-02-23 | 1 | -1/+1 |
| | |||||
* | Fix inconsistent naming for services that delete things | dixpac | 2017-02-08 | 1 | -1/+1 |
| | | | | | | * Changed name of delete_user_service and worker to destroy * Move and change delete_group_service to Groups::DestroyService * Rename Notes::DeleteService to Notes::DestroyService | ||||
* | Merge branch 'fix-guest-access-posting-to-notes' into 'security' | Robert Speicher | 2017-01-23 | 1 | -10/+16 |
| | | | | | Prevent users from creating notes on resources they can't access See https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2054 | ||||
* | Parameter already enforced via grapeclean-api-params | Robert Schilling | 2016-12-28 | 1 | -2/+0 |
| | |||||
* | Use the pagination helper in the APIuse-pagination-helper | Robert Schilling | 2016-12-04 | 1 | -1/+3 |
| | |||||
* | Grapify the notes APIgrapify-notes-api | Robert Schilling | 2016-11-15 | 1 | -67/+57 |
| | |||||
* | Fix API notes endpoint when posting only emoji | Z.J. van de Weg | 2016-09-16 | 1 | -4/+4 |
| | |||||
* | Sort API endpoints and implement feedback | Z.J. van de Weg | 2016-06-17 | 1 | -1/+1 |
| | |||||
* | Move note helper method to notes entity fileissue_17302 | Felipe Artur | 2016-05-17 | 1 | -2/+8 |
| | |||||
* | Fix single note api request | Felipe Artur | 2016-05-16 | 1 | -5/+5 |
| | |||||
* | Fix notes API calls symbol convertions | Felipe Artur | 2016-05-10 | 1 | -3/+3 |
| | |||||
* | Fix api leaking notes when user is not authorized to read noteable | Felipe Artur | 2016-05-09 | 1 | -13/+18 |
| | |||||
* | Allow back dating notes on creation | Michael Greene | 2016-04-13 | 1 | -0/+5 |
| | |||||
* | Adapt tests to new testing guidelines | Robert Schilling | 2016-04-12 | 1 | -0/+2 |
| | |||||
* | Fix code review issues | Robert Schilling | 2016-04-12 | 1 | -4/+2 |
| | |||||
* | Delete notes via API | Robert Schilling | 2016-04-12 | 1 | -0/+17 |
| | |||||
* | Ensure the API doesn't return notes that the current user shouldn't see | Rémy Coutable | 2016-01-13 | 1 | -2/+19 |
| | |||||
* | Show who last edited a comment if it wasn't the original author | Douwe Maan | 2015-07-30 | 1 | -7/+5 |
| | |||||
* | Add a message when unable to save an object through api. | Marin Jankovski | 2015-01-07 | 1 | -1/+1 |
| | |||||
* | Forward the messages in api response. | Marin Jankovski | 2014-12-30 | 1 | -1/+1 |
| | |||||
* | Implemented notes (body) patching in API. | uran | 2014-12-25 | 1 | -0/+33 |
| | |||||
* | Fix notes creation via API | Dmitriy Zaporozhets | 2014-06-18 | 1 | -5/+8 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Wipe wall notes feature | Dmitriy Zaporozhets | 2014-06-13 | 1 | -49/+0 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Remove thread vars usage from API notes and mr's | Dmitriy Zaporozhets | 2014-06-10 | 1 | -22/+18 |
| | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | ||||
* | Fixing unsafe use of Thread.current variable :current_user | Angus MacArthur | 2013-10-16 | 1 | -18/+22 |
| | |||||
* | Refactor API classes. So api classes like Gitlab::Issues become API::Issues | Dmitriy Zaporozhets | 2013-05-14 | 1 | -1/+1 |
| | |||||
* | load notes for wall via api | Dmitriy Zaporozhets | 2013-03-19 | 1 | -0/+4 |
| | |||||
* | API: fixed adding a note | Sebastian Ziebell | 2013-02-27 | 1 | -1/+1 |
| | | | | Now the correct attribute is checked if it's available or not. Also fixed a test. | ||||
* | API: extracted helper method to validate required parameters, code clean up | Sebastian Ziebell | 2013-02-27 | 1 | -3/+2 |
| | | | | | | Added a helper method to check if required parameters are given in an API call. Can be used to return a `400 Bad Request` return code if a required attribute is missing. Code clean up and fixed tests. |