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

Article VII:4 and VII:5 #26

Merged
merged 37 commits into from
Jul 16, 2020
Merged

Article VII:4 and VII:5 #26

merged 37 commits into from
Jul 16, 2020

Conversation

ColinMaudry
Copy link
Contributor

(includes commits from VII:3)

I'm not happy with the mapping of the date publication of the notice of intended procurement, in VII:4. I think a Milestone would have been more suitable, but I found no suitable milestone type code.

In the EU profile we used tender.communication.futureNoticeDate. This property was created for the context of the TED forms, so the semantics might a bit loose for the GPA context. What do you think?

@ColinMaudry
Copy link
Contributor Author

The mapping goes faster than expected. I wll keep on committing on this PR until EoD.

@ColinMaudry ColinMaudry changed the title Article VII:4 and VII:5 WIP: Article VII:4 and VII:5 Jun 2, 2020
@ColinMaudry
Copy link
Contributor Author

Are you sure the guidance for VII:2(k) is good? It doesn't seem to match the text.

@ColinMaudry
Copy link
Contributor Author

I'll wait for your feedback and the validation of the patterns I used before mapping further.

@ColinMaudry ColinMaudry changed the title WIP: Article VII:4 and VII:5 Article VII:4 and VII:5 Jun 4, 2020
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
@jpmckinney
Copy link
Member

Are you sure the guidance for VII:2(k) is good? It doesn't seem to match the text.

The guidance for VII:2(k) on this page looks fine here: https://standard.open-contracting.org/profiles/gpa/master/en/gpa/ You fixed it in #25.

@ColinMaudry
Copy link
Contributor Author

Done down to X:11 included.

@ColinMaudry
Copy link
Contributor Author

ColinMaudry commented Jun 17, 2020

I'm tempted to add

  • Enter the Web address where the document can be viewed or download in its url

whenever a Document is added, even if the document URL is not mentioned in the GPA article. That goes beyond the instructions of the GPA, but Document without a URL is a bit useless.

What do you think?

Copy link
Member

@jpmckinney jpmckinney left a comment

Choose a reason for hiding this comment

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

I'm tempted to add

  • Enter the Web address where the document can be viewed or download in its url

whenever a Document is added, even if the document URL is not mentioned in the GPA article. That goes beyond the instructions of the GPA, but Document without a URL is a bit useless.

What do you think?

We can have specific guidance as you suggest, or add a bullet suggesting to fill in any other known information for the document: https://standard.open-contracting.org/1.1/en/schema/reference/#document

docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
Comment on lines +155 to +158
* Add *a list and brief description of any conditions for participation of suppliers* to `tender/eligibilityCriteria`
* If *requirements are included in tender documentation that is made available to all interested suppliers*:
* For each document, add a `Document` object to the `tender/documents` array
* Set its `documentType` to 'eligibilityCriteria'
Copy link
Member

Choose a reason for hiding this comment

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

I think this clause refers to selection criteria, similar to those in TED forms: open-contracting/standard#901 As I remember, TED forms had a few different ways of expressing such criteria.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You mentioned on the call that certain keywords lead you to think this clause is related to selection criteria, but

  • selection criteria are clearly mentioned in the next clause VII:2(k)
  • eligibility criteria seem to be mentioned nowhere else (although the GPA may not mention them at all)

Do I repeat the guidance given in the next clause?

I just find that the GPA having two neighbour clauses about selection criteria is unlikely.

Copy link
Member

Choose a reason for hiding this comment

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

#30

docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Show resolved Hide resolved
@jpmckinney
Copy link
Member

@ColinMaudry Let's resolve my comments and then merge this PR. It's become too long to review easily if we add any new changes :)

docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
docs/gpa.md Outdated Show resolved Hide resolved
@ColinMaudry
Copy link
Contributor Author

I'm tempted to add

  • Enter the Web address where the document can be viewed or download in its url

whenever a Document is added, even if the document URL is not mentioned in the GPA article. That goes beyond the instructions of the GPA, but Document without a URL is a bit useless.
What do you think?

We can have specific guidance as you suggest, or add a bullet suggesting to fill in any other known information for the document: https://standard.open-contracting.org/1.1/en/schema/reference/#document

Done in e06dd10

@ColinMaudry
Copy link
Contributor Author

Did you review XIII:2 ?

@jpmckinney
Copy link
Member

Did you review XIII:2 ?

I skimmed it. I think after we merge, I'll re-read the full, rendered page just to double check.

Copy link
Contributor Author

@ColinMaudry ColinMaudry left a comment

Choose a reason for hiding this comment

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

Forget XIII:2 for this PR, I'll improve it in a dedicated PR.

Comment on lines +155 to +158
* Add *a list and brief description of any conditions for participation of suppliers* to `tender/eligibilityCriteria`
* If *requirements are included in tender documentation that is made available to all interested suppliers*:
* For each document, add a `Document` object to the `tender/documents` array
* Set its `documentType` to 'eligibilityCriteria'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

You mentioned on the call that certain keywords lead you to think this clause is related to selection criteria, but

  • selection criteria are clearly mentioned in the next clause VII:2(k)
  • eligibility criteria seem to be mentioned nowhere else (although the GPA may not mention them at all)

Do I repeat the guidance given in the next clause?

I just find that the GPA having two neighbour clauses about selection criteria is unlikely.

@ColinMaudry ColinMaudry merged commit 61c7aed into master Jul 16, 2020
@ColinMaudry ColinMaudry deleted the VII-4-5 branch July 16, 2020 09:48
@ColinMaudry
Copy link
Contributor Author

I have merged so that I can proceed with the next PRs.

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.

None yet

2 participants