Skip to content

Conversation

dpup
Copy link
Member

@dpup dpup commented Oct 28, 2019

What's this PR do?

Adds the AddSuggestion endpoint to the API

Where should the reviewer start?

Diff

Any background context you want to provide?

There's also some changes that were required for the NPM push that I forgot to
commit.

What are the relevant tickets?

Screenshots (if appropriate)

What gif best describes this PR or how it makes you feel?

Definition of Done:

  • Code is easy to understand and conforms with Prettier & eslint configs
  • Incomplete code is marked with TODOs
  • Code is suitably instrumented with logging and metrics
  • Documentation has been updated as appropriate
  • Manifest has been updated and version incremented correctly
  • OWASP Top 10 have been considered

@dpup dpup requested a review from chrisforrette October 28, 2019 23:02
@dpup
Copy link
Member Author

dpup commented Nov 5, 2019

@chrisforrette ping :)

@dpup dpup merged commit 3500027 into master Nov 7, 2019
@dpup dpup deleted the dpup/docs branch November 7, 2019 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants