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

Make tender.selectionCriteria and tender.exclusionGround objects with .description fields #1607

Closed
duncandewhurst opened this issue Apr 17, 2023 · 5 comments · Fixed by #1674
Assignees
Labels
Schema: Fields Relating to adding or deprecating fields in the JSON Schema
Milestone

Comments

@duncandewhurst
Copy link
Contributor

duncandewhurst commented Apr 17, 2023

In OCDS 1.2, there’s a string tender.selectionCriteria field (previously discussed in #1120). For OCDS 1.1's EU profile, we created the selection criteria extension, which uses the same field name, but breaks it down into an object with a .description field and a .criteria array for structured data.

Similarly, in OCDS 1.2, there's a string tender.exclusionGrounds field (previously discussed in #901). For OCDS 1.1's eForms profile, we need to create an extension for structured data on exclusion grounds.

The proposal for this issue is to take the approach suggested in #901 (comment) of making both tender.selectionCriteria and tender.exclusionGround objects with .description fields. Extensions can then add other fields to the objects for structured data.

{
  "tender": {
    "selectionCriteria": {
      "description": "unstructured data",
      "criteria": [
        // structured data
      ]
    },
    "exclusionGrounds": {
      "description": "unstructured data",
      "criteria": [
        // structured data
      ]
    }
  }
}
@duncandewhurst duncandewhurst added the Schema: Fields Relating to adding or deprecating fields in the JSON Schema label Apr 17, 2023
@duncandewhurst duncandewhurst added this to the 1.2.0 milestone Apr 17, 2023
@jpmckinney
Copy link
Member

Agree!

@odscjen
Copy link
Contributor

odscjen commented Jan 25, 2024

Just tried adding a field to the SelectionCriterion in the selection criteria field and it's giving an error "unexpectedly overwrites /definitions/Tender/properties/selectionCriteria/title /home/jen/extensions/ocds_selectionCriteria_extension/release-schema.json".

@jpmckinney you've assigned yourself to this issue but I'm guessing it's not that high on your to do list? Can I create a PR to address it and so unblock open-contracting-extensions/ocds_selectionCriteria_extension#8

@jpmckinney
Copy link
Member

I forgot this issue existed 😅

I created open-contracting/ocds-extensions#221 about the conflict. I think you can just delete Tender.selectionCriteria from the extension, as it's only used by the old EU profile as suggested there.

@jpmckinney
Copy link
Member

jpmckinney commented Jan 25, 2024

That said, feel free to also work on this issue – but if you want to unblock open-contracting-extensions/ocds_selectionCriteria_extension#8, changing the extension as suggested above is the quickest route.

@odscjen
Copy link
Contributor

odscjen commented Jan 26, 2024

but if you want to unblock open-contracting-extensions/ocds_selectionCriteria_extension#8, changing the extension as suggested above is the quickest route.

I'm assuming that's referring to the suggestion

I think you can just delete Tender.selectionCriteria from the extension, as it's only used by the old EU profile as suggested there.

so suggestion 3 in open-contracting/ocds-extensions#221?

I'll move discussion of the options to open-contracting/ocds-extensions#221 but simply removing Tender.selectionCriteria from the extension might not be a suitable unblock for the blocked PR as the new field being added to Criterion might be needed at tender level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Schema: Fields Relating to adding or deprecating fields in the JSON Schema
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants