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

Allow multiple default aid type #498

Merged
merged 4 commits into from
Sep 25, 2018
Merged

Conversation

amy-silcock
Copy link
Contributor

Linked to issue: IATI/IATI-Codelists#169

.. literalinclude:: ../../activity-standard-example-annotated.xml
:language: xml
:start-after: <!--default-aid-type starts-->
:end-before: <!--default-aid-type ends-->

The vocabulary is used to declare which AidType list is in use. If this is omitted, then IATI assumes an AidType of 1 (OECD DAC).
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is omitted, then IATI assumes an AidType of 1

This is about Aid Type Vocabulary, not Aid Type. If omitted, an Aid Type Vocabulary of 1 is assumed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for spotting this @andylolz

@PetyaKangalova
Copy link
Contributor

@samuele-mattiuzzo I have made the change. Could you please review and merge?

Keeping this consistent with aid-type by specifying that code 1 is OECD DAC
@samuele-mattiuzzo
Copy link
Contributor

@PetyaKangalova will do!

.. literalinclude:: ../../activity-standard-example-annotated.xml
:language: xml
:start-after: <!--default-aid-type starts-->
:end-before: <!--default-aid-type ends-->

The vocabulary is used to declare which AidType list is in use. If omitted, an AidType Vocabulary of 1 (OECD DAC) is assumed.
Copy link
Contributor

Choose a reason for hiding this comment

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

“AidTypeVocabulary” and “Aid Type Vocabulary” are used interchangeably in the docs (ref). Not really keen on this new version (“AidType Vocabulary”).

Copy link

@markbrough markbrough left a comment

Choose a reason for hiding this comment

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

Clarify that default-aid-type should not be used more than once per vocabulary.

<default-aid-type code="A01" vocabulary="1" />


The default-aid-type element can be repeated in any iati-activity.

Choose a reason for hiding this comment

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

I think this should be clarified to something like:

The default-aid-type element must not be used more than once for each vocabulary.


From the discussion on Discuss:

Also, I think this should be done in a new Rule, because at present the (automatically generated) rules on that page are the most confusing (and inaccurate) thing:

grafik

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @markbrough your suggested wording, but with a should, is already in the activity schema. We are working on trying to get this shown on the website: https://github.com/IATI/IATI-Schemas/blob/version-2.03/iati-activities-schema.xsd

Choose a reason for hiding this comment

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

Oh excellent, thanks @amy-silcock! Yes agree that is a better approach :)

Copy link
Contributor

Choose a reason for hiding this comment

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

We are working on trying to get this shown on the website

I suppose you mean this line. Quite difficult to figure out what happened with this. It was added to the wrong branch in IATI/IATI-Schemas#418, then reverted in IATI/IATI-Schemas#420… I guess it was re-added recently, and hasn’t actually been deployed. So I’d suggest deploying v2.03.

As usual, I think auto-deploying the reference site from github would be a really sensible step that could save some headaches. See: IATI/IATI-Standard-SSOT#164.

@Ocre42 Ocre42 merged commit 08379df into version-2.03 Sep 25, 2018
@Ocre42 Ocre42 deleted the allow-multiple-default-aid-type branch September 25, 2018 10:01
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.

6 participants