-
Notifications
You must be signed in to change notification settings - Fork 9
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
Review the numbering of all newly numeric codes #48
Comments
@bill-anderson Could you review the new numbering used for each of these codelists? |
Another proposal might be to ensure that the default value for a Vocabulary is always numbered 1. ie. that SectorVocabulary has code 1 as 5 digit DAC codes. Note that SectorVocabulary probably also has too many codes on it, see #43 |
Agree with 1 = default value. Not fussed about others. I'm sure @stevieflow will have a grand plan |
The concern with |
Renumbered SectorVocabulary in this commit. I've glanced through the other codelists, and the only other potential issue I spot is OrganisationRole now has Accountable as code 1 (because I numbered them alphabetically), but Accountable is actually one of the less used codes. @stevieflow I agree that ActivityDate might also be useful numbered more chronologically. How about:
|
ok thanks Agree with the new numbers for @bill-anderson any thoughts on |
Looking at http://dev.iatistandard.org/201/codelists/OrganisationRole/ Could these be in a different order, perhaps?: Funcing |
Could be. I agree with whatever you think is best |
Definitive answer! 1 Funding seems more in line with "flow" than: 1 Accountable ? @jonihillman @iatisupport ? |
Definitive answer looks about right to me! |
Do we take the revised order as a change for @Bjwebb to implement therefore? |
As he said - "I agree with whatever you think is best" |
Assigned to @Bjwebb to implement |
@Bjwebb I have created new Issues for the three codelists that need renumbering Aside from that, the only comment I can see in this Issue that may need some action is #48 (comment) - is this correct (or can we close this?)? |
I think this issue can now be closed. |
Steven made a comment about the ActivityDateType numbering, but we should check the other codelists too:
http://dev.iatistandard.org/201/codelists/ActivityDateType/
http://dev.iatistandard.org/201/codelists/GazetteerAgency/
http://dev.iatistandard.org/201/codelists/OrganisationRole/
http://dev.iatistandard.org/201/codelists/TransactionType/
http://dev.iatistandard.org/201/codelists/SectorVocabulary/
(Issue for the change to numeric: #25)
The text was updated successfully, but these errors were encountered: