Skip to content
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

Various updates in generated files #819

Merged
merged 44 commits into from
Feb 5, 2024
Merged

Conversation

bertrand-lorentz
Copy link
Contributor

@bertrand-lorentz bertrand-lorentz commented Feb 2, 2024

Lots of updates for SDK 1.11, in most of the generated files.
Please see the individual commits.

The failure of the SDK Analyzer is due to invalid characters in translations texts. Correction for this is in progress.

Update corresponding schematron rules, and affected validation reports.
In Schematron, phase identifiers should not start with a digit, so add
the prefix "eforms-" to all phase identifiers: "eforms-16".
So when executing the schematron, you must also add this prefix when you
specify the phase to execute based on the notice subtype.
Also update corresponding schematron rules.
Also update corresponding Schematron rules.
Also add corresponding Schematron.
Values should be unique in the group of lots, not across the whole
notice. This is not supported currently, so remove the assert.

Also remove from Schematron rules.
Winner Decision (BT-1451) can be on the same day the notice is submitted
(BT-05) and the contract is signed (BT-145).

Also update corresponding Schematron rules.
Also update corresponding schematron rules.
Also update corresponding Schematron rules.
The fields BT-22-Lot and BT-22-LotsGroup are now optional.
Add rules to check that fields that contain structured values (id, code)
do not have leading or trailing whitespace.

Update affected validation reports.

See also discussion #779.
The country codelist now has the same content as eforms-country, so we
don't need this tailored codelist anymore.

Also update references to this codelist in fields.json.
CPV codes had no translations in Croatian, as they are not in the
source used.

Add those translations in the corresponding codelists
files, and in the translation file code_hr.xml.
Change constraints to allow change notices with "cancel-intent" to be
submitted after the tender deadline has passed.
For asserts that calculate a sum of several values, do not require a
strict equality but allow some approximation. This is to avoid
validation failures when some values have been rounded up or down.
The RELAX NG schema in the Schematron SIO standard requires some
elements to be in a certain order: title, ns, etc.
Move a few elments to avoid any errors when validating against this
schema.

This has no effect on the execution of the Schematron, implementations
seems less strict than the schema.
Copy link
Contributor

@rouschr rouschr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that this commit is quite old, note that I synchronized with their new official CPV codelist that has these labels recently, in theory the labels should be the same.

@bertrand-lorentz
Copy link
Contributor Author

I see that this commit is quite old, note that I synchronized with their new official CPV codelist that has these labels recently, in theory the labels should be the same.

Any changes since the commits in this PR will be reflect in future commits/PR.

@bertrand-lorentz bertrand-lorentz merged commit 1ef1dea into develop Feb 5, 2024
0 of 2 checks passed
@bertrand-lorentz bertrand-lorentz deleted the various-updates branch February 5, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants