-
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
Edit exception message when editing certain fields of a catalogue item with children #418
Edit exception message when editing certain fields of a catalogue item with children #418
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #418 +/- ##
========================================
Coverage 97.87% 97.87%
========================================
Files 41 41
Lines 1504 1504
========================================
Hits 1472 1472
Misses 32 32 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
General comment could you perhaps list the fields from CATALOGUE_ITEM_WITH_CHILD_NON_EDITABLE_FIELDS
so it's kept up to date with changes. Also I just noticed we also have exactly the same sort of error message for the catalogue category edit could you update these as well so they are consistent.
- with changes to pass tests
…353 - and test changes to pass
The linter check is failing, but I'm pretty sure all the lines I added are under 120. So is this failing because of previously written lines in the files I edited? |
Could be that your local version of black needs updating or vscode isn't autoformatting correctly. You can use |
|
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. I added a corresponding front end issue here ral-facilities/inventory-management-system#1136.
Description
Edit the returned exception message to make it clearer when trying to update protected fields on a catalogue item or catalogue category that has a child element.
Testing instructions
Add a set up instructions describing how the reviewer should test the code
Agile board tracking
Resolves #353