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

Review OCDS Kit's OC4IDS transform and update mapping #210

Closed
duncandewhurst opened this issue Feb 14, 2020 · 3 comments · Fixed by #294
Closed

Review OCDS Kit's OC4IDS transform and update mapping #210

duncandewhurst opened this issue Feb 14, 2020 · 3 comments · Fixed by #294
Assignees
Labels
documentation This issue relates to the documentation
Milestone

Comments

@duncandewhurst
Copy link
Contributor

duncandewhurst commented Feb 14, 2020

Review the logic used in the OCDS-OC4IDS transform for each mapping and update the mapping table in the OC4IDS docs as required.

In particular, improve the documentation for the project sector mapping to explain what is left to the reader.

@duncandewhurst duncandewhurst added the documentation This issue relates to the documentation label Feb 14, 2020
@duncandewhurst
Copy link
Contributor Author

In particular, improve the documentation for the project sector mapping to explain what is left to the reader.

At the same time we can update the OC4IDS mapping column to explain that a code from the OC4IDS sector codelist should be used, since the description from the IDS mentions developing a list.

@jpmckinney jpmckinney changed the title Review OCDSkit's OC4IDS transform and update mapping Review OCDS Kit's OC4IDS transform and update mapping Sep 10, 2020
@duncandewhurst duncandewhurst self-assigned this Jan 28, 2021
@pindec pindec self-assigned this Jan 29, 2021
pindec added a commit that referenced this issue Feb 1, 2021
@pindec
Copy link
Contributor

pindec commented Feb 3, 2021

See also ocdskit convert-to-oc4ids issue #165

@pindec pindec mentioned this issue Feb 24, 2021
4 tasks
@duncandewhurst
Copy link
Contributor Author

For posterity, @pindec's notes from reviewing the mapping are in a Google Doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This issue relates to the documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants