Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

Add API support for deleting a comment #1114

Closed
kumar303 opened this issue Oct 9, 2019 · 1 comment · Fixed by #1115
Closed

Add API support for deleting a comment #1114

kumar303 opened this issue Oct 9, 2019 · 1 comment · Fixed by #1115
Assignees
@kumar303
Labels
priority:p3 state:verified_fixed
Milestone
2019.10.17

Comments

@kumar303
Copy link
Contributor

kumar303 commented Oct 9, 2019

Add API changes needed to delete a comment. See #714 for the UI.

For QA: this change will affect all API calls so please do a rough sanity check on any feature involving API calls (browsing, comparing, viewing linter messages, log in, log out)
@kumar303 kumar303 added this to the 2019.10.17 milestone Oct 9, 2019
@kumar303 kumar303 self-assigned this Oct 9, 2019
@kumar303 kumar303 mentioned this issue Oct 9, 2019
Merged
@ioanarusiczki
Copy link

@kumar303
I verified on dev with FF69(Win10)
Browse
https://code.addons-dev.allizom.org/en-US/browse/492486/versions/1529211/
Compare
https://code.addons-dev.allizom.org/en-US/compare/492486/versions/1529211...1688395/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kumar303 kumar303

Labels
priority:p3 state:verified_fixed
Projects
None yet
Milestone
2019.10.17
Development

Successfully merging a pull request may close this issue.

Add deleteComment() API and support DELETE responses better kumar303/addons-code-manager
2 participants
@kumar303 @ioanarusiczki