Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Add Report state change, assignee update, and comment functionality #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bored-engineer
Copy link
Contributor

HackerOne previously added endpoints for creating comments, updating a report's assignee, and updating a report's state:

September 21st, 2016: added ability to assign users and groups to a report.
November 2nd, 2016: added ability to change the state of a report object and added ability to post internal and public comments.

This pull request adds support for these endpoints via additional methods on the ReportService struct. Since there are the first state-changing endpoints with POST bodies changes were made to h1.go to add support for request bodies. Additionally this was the first time a struct (Activity) had to be JSON marshalled so some changes have been made there. I'm open to suggestions about the best method to do this since I'm not thrilled with the current implementation but it does work...

@bored-engineer
Copy link
Contributor Author

This fixes #9, #10, and #11.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants