-
Notifications
You must be signed in to change notification settings - Fork 8
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
[MODORDERS-964] Update acq-models #806
Conversation
@alb3rtino we also need to create PR with updating models for mod-gobi repository, because it also uses the same schemas |
checked that also need to create similar PRs to update references in mod-invoice, mod-invoice-storage |
Taking a closer look at this, as schemas for
|
I've updated to provided interface versions to
and the required interface versions to
This requires a merge of MODORDSTOR PR#374 first, so i've added the |
c22d5b4
to
dd1c61f
Compare
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
@SerhiiNosko I've made updates to the interface versions. Could you please review them? I'm uncertain whether the updated required interfaces require a module major version increase or not. |
@@ -4,7 +4,7 @@ | |||
"provides": [ | |||
{ | |||
"id": "orders", | |||
"version": "12.0", | |||
"version": "12.1", |
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.
As I remember if we increase from 12.0 to 12.1 - Okapi will not throw an error, even if in mod-gobi required version is 12.0 https://github.com/folio-org/mod-gobi/blob/master/descriptors/ModuleDescriptor-template.json#L157
If we for example we increased from 12.0 to 13.0 and in mod-gobi remain 12.0 - Okapi will throw an error during enabling the module.
But you can check my words using karate tests(acquisition folder in folio-integration-test) or just locally.
Basically we not always follow the rule to update minor version when adding new field, but in case when you increased it also would be a good practice to increase in these modules as well:
https://github.com/folio-org/mod-invoice/blob/master/descriptors/ModuleDescriptor-template.json#L664
https://github.com/folio-org/mod-gobi/blob/master/descriptors/ModuleDescriptor-template.json#L157
https://github.com/folio-org/mod-ebsconet/blob/master/descriptors/ModuleDescriptor-template.json#L6
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.
Basically I am ok to keep major versions for example in mod-gobi 12.0 and in orders increase to 12.1. It should work
Purpose
This PR updates
acq-models
to include thecustomFields
attribute to purchase orders and purchase order lines. This is part of the custom fields integration. See MODORDERS-964.Prerequisite
Related