-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Partially edit catalogue categories #62
Conversation
ff5f256 addresses the change requested in #45 (comment). |
Supplying a list of catalogue item properties in a |
Can you add error checking for the Manufacturer URL to test if it's a valid URL? I've had to add this in the frontend because if the URL is invalid and a user clicks on it, it appends the invalid URL to the current URL. This would also needs to be done in #67. |
Addressed in 7d82c8d
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Description
This PR creates a catalogue items endpoint (accessible at
/v1/catalogue-categories/{catalogue_category_id}
) to whichPATCH
requests can be sent to partially update a catalogue category by ID. The endpoint requires only the fields that have changed to be supplied. For example, if thename
needs to be changed then only thename
field and corresponding value should be supplied.Testing instructions
Agile board tracking
closes #3