Annotation: per-nodule notes should actually be PATCHed and saved

drivendataorg / concept-to-clinic

ALCF Concept to Clinic Challenge

https://concepttoclinic.drivendata.org/

MIT License

367 stars 147 forks source link

Closed isms closed 6 years ago

isms commented 6 years ago

Manipulating these should be PATCHed to the backend and saved in the appropriate model instance:

No-op

Serhiy-Shekhovtsov commented 6 years ago

I'd like to take this one.

Serhiy-Shekhovtsov commented 6 years ago

@isms, I can see that we are using PATCH for updating the candidate. Is there a reason for using POST here?

isms commented 6 years ago

@Serhiy-Shekhovtsov no, not at all, PATCH is the right verb 👍

isms commented 6 years ago

(issue updated to say PATCH - thanks Serhiy)

Serhiy-Shekhovtsov commented 6 years ago

@isms the issue is solved in PR #286