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

December 2017 DAC codelist update #231

Merged
merged 4 commits into from
Mar 21, 2018
Merged

December 2017 DAC codelist update #231

merged 4 commits into from
Mar 21, 2018

Conversation

andylolz
Copy link
Contributor

@andylolz andylolz commented Jan 6, 2018

DAC codelists were updated in December. I think this PR includes all of the changes.

NB this PR was generated using #172.

Part of December 2017 OECD DAC update
Part of December 2017 OECD DAC update
Part of December 2017 OECD DAC update
Part of December 2017 OECD DAC update
@andylolz andylolz changed the title December 2017 dac codelist update December 2017 DAC codelist update Jan 6, 2018
@hayfield
Copy link
Contributor

@amy-silcock Is there an associated PR to update the Changelog?

@andylolz
Copy link
Contributor Author

There are some January 2018 changes now. Is it worth adding those to this PR, too?

@andylolz
Copy link
Contributor Author

andylolz commented Mar 6, 2018

It looks like these changes have been approved by @amy-silcock 👍. Is there a plan to merge this? (The pull request and discuss post are now three months old).

Once this is sorted, I’ll send a PR for the end-of-January DAC codelist updates.

@amy-silcock amy-silcock requested review from hayfield, dalepotter and allthatilk and removed request for PetyaKangalova March 21, 2018 17:04
@allthatilk allthatilk merged commit 87f5ab7 into IATI:master Mar 21, 2018
@andylolz andylolz deleted the december-2017-dac-codelist-update branch March 21, 2018 17:26
@andylolz
Copy link
Contributor Author

🎉 🎉 🎉

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