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

Importing data and schema configuration from standard JSON file #435

Open
kennethbruskiewicz opened this issue Apr 25, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@kennethbruskiewicz
Copy link
Collaborator

Riffing off of this: #431 (comment)

given a data structure like:

{
    "schema": "https://www.example.com/GRDI",
    "version": "10.0.0",
    "GRDI_Sample": [ /* list items*/ ],
    "AMR_Test": [ /* list items*/ ]
}

we can imagine using the "in_language" property like

{
    "schema": "https://www.example.com/GRDI",
    "version": "10.0.0",
    "in_language": "fr",
    "GRDI_Sample": [ /* list items*/ ],
    "AMR_Test": [ /* list items*/ ]
}

This closes the import-export loop for the application, allowing users to save where they left off and have the application state restored to normal. It also means that when the file is passed around, it can self-document what the language content is.

This seems good-enough, with the only possible extension being a self-referential or error-checkable hash that can ensure file integrity.

@kennethbruskiewicz kennethbruskiewicz added the enhancement New feature or request label Apr 25, 2024
@kennethbruskiewicz kennethbruskiewicz self-assigned this Apr 25, 2024
@ddooley
Copy link
Collaborator

ddooley commented Apr 25, 2024

Good, but as noted in other thread the top-level objects of this should match directly the Container spec in the schema I think, so plural GRDI_Samples, etc...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants