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

Replace hand-made json validator by validateJSON utility #1052

Open
thamara opened this issue Jan 8, 2024 · 2 comments
Open

Replace hand-made json validator by validateJSON utility #1052

thamara opened this issue Jan 8, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@thamara
Copy link
Owner

thamara commented Jan 8, 2024

In #946 a new formal json validator was added. The task here is to replace the current way of validating the json file in js/import-export.js by the one implemented in js/validate-json.js.

@thamara thamara added the enhancement New feature or request label Jan 8, 2024
@nehagupta1504
Copy link

@thamara can you point to me the line no. in js/import-export.js which you are asking to add validation or replacing the older way

@tupaschoal
Copy link
Collaborator

@nehagupta1504 to me it looks like the validEntry function should be replaced.

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

3 participants