-
Notifications
You must be signed in to change notification settings - Fork 5
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
Allow barcode to be cleared out #3067
Conversation
15eafe9
to
f837646
Compare
@andrewjbtw this PR is not deployed anywhere, so it is not testable |
f837646
to
77f3d21
Compare
Ah, ok. Then I'll ignore the error for now. |
43cfc2f
to
ece84f3
Compare
@@ -16,7 +16,7 @@ def initialize(model, change_set) | |||
|
|||
def update | |||
updated = model | |||
updated = update_identification(updated) if changed?(:source_id) || changed?(:catkey) || changed?(:barcode) | |||
updated = update_identification(updated) if changed?(:source_id) || changed?(:catkey) |
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.
collections don't have barcode
Currently our openapi.yaml isn't going to support |
ece84f3
to
490b0d5
Compare
Why was this change made?
Fixes #3065
How was this change tested?
Which documentation and/or configurations were updated?