forem icon indicating copy to clipboard operation
forem copied to clipboard

Reactions API: Delete reaction

Open joshpuetz opened this issue 3 years ago • 1 comments

Reactions API requirements: https://docs.google.com/document/d/1O8YG8moIMUHi5_A-VHVWDmdIaYeHFEpeBEFGRyxc-PQ/edit#bookmark=id.z6jwzd5e5y70

Description: create an endpoint for deleting a Reaction.

Authorization: api key

Permissions: limited to users with Admin or Super Admin role

Path: DELETE /api/reactions/:id

Body params: none

Return body: none

Return codes: 204 No Content 401 Unauthorized 403 Forbidden 404 Not Found 429 Too Many Requests

joshpuetz avatar Aug 02 '22 19:08 joshpuetz

Thanks for the issue, we will take it into consideration! Our team of engineers is busy working on many types of features, please give us time to get back to you.

Feature requests that require more discussion may be closed. Read more about our feature request process on forem.dev.

To our amazing contributors: issues labeled bug are always up for grabs, but for feature requests, please wait until we add a ready for dev before starting to work on it.

To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please ping @forem-team. The OSS Community Manager or the engineers on OSS rotation will follow up.

For full info on how to contribute, please check out our contributors guide.

github-actions[bot] avatar Aug 02 '22 19:08 github-actions[bot]

Based on this conversation and this one we've decided to not implement the "Delete" action, and instead just have the "Create" action change to POST to /reactions/toggle. This covers both a create and delete together.

Ridhwana avatar Aug 23 '22 10:08 Ridhwana