Skip to content
This repository has been archived by the owner on Oct 25, 2022. It is now read-only.

Adding DCOI_StrategicPlans_fy2017Schema.json #22

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

vasili4
Copy link

@vasili4 vasili4 commented Jan 29, 2017

I have applied the latest UI changes made in schema.md d8291be , to the schemaexamples/DCOI_StrategicPlans_fy2016Schema.json and saved it as a new schemaexamples/DCOI_StrategicPlans_fy2017Schema.json , changed schema version and title to fy2017.

List of changes:

  • Added DCOI_StrategicPlans_fy2017Schema.json to schemaexamples/
  • Changed link "DCOI Strategic Plan JSON Schema" to the latest fy2017 schema file.
  • Removed redundant " schema" word from the end of every field title.

I'm planning to use same schema to update https://datacenters.cio.gov/json-conversion-tool/

@vasili4
Copy link
Author

vasili4 commented Jan 29, 2017

related #21

@BaconMichelle
Copy link
Contributor

@alex-perfilov-reisys
Thanks, Alex.
I think Sarah A asked me not to change the file name in the past because of versioning concerns. Can you triple check that all references to the schema have been updated in the Java tool? Thank you!

@vasili4
Copy link
Author

vasili4 commented Feb 7, 2017

@BaconMichelle , I'm not sure which Java tool should I check. If you meant JSON Conversion Tool, which is written in JavaScript, then it does not have any references to the schema. It uses a local copy instead.

Also, I did not remove DCOI_StrategicPlans_fy2016Schema.json from repo. I've added a new fy2017 schema in same folder instead.

@BaconMichelle
Copy link
Contributor

@alex-perfilov-reisys
Gotcha. Understood that nothing has been removed. Ideally, if I made a change to the schema file without renaming it, the tool would automatically update. Is that possible? Or no...

@vasili4
Copy link
Author

vasili4 commented Feb 7, 2017

@BaconMichelle , that might be doable. But it also has some sample data which should also be in-sync

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants