Skip to content
Snippets Groups Projects

API: Moved `DELETE /projects/:id/star` to `POST /projects/:id/unstar`

Merged username-removed-2900 requested to merge api-star-restful into master
All threads resolved!

What does this MR do?

Moves the unstar API endpoint from DELETE to POST to make it more restful.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/28328

Merge request reports

Pipeline #6581586 passed with warnings

Pipeline passed with warnings for 039c6d60 on api-star-restful

Test coverage 93.50% (0.07%) from 1 job

Merged by username-removed-128633username-removed-128633 8 years ago (Feb 20, 2017 5:04pm UTC)

Loading

Pipeline #6585641 passed with warnings

Pipeline passed with warnings for 5f0e4619 on master

Test coverage 93.22% (0.07%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • username-removed-2900 resolved all discussions

    resolved all discussions

  • added 1 commit

    • 039c6d60 - API: Moved DELETE /projects/:id/star to POST /projects/:id/unstar

    Compare with previous version

  • @rymai I updated the MR.

  • username-removed-2900 added ~164274 ~889916 labels

    added ~164274 ~889916 labels

  • @razer6 Awesome work, thank you! :heart:

  • Toon Claes mentioned in merge request !11449 (merged)

    mentioned in merge request !11449 (merged)

  • Please register or sign in to reply
    Loading