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

Pre-parse JSON for proper format and all required elements #17

Open
fuddster opened this issue Feb 12, 2022 · 1 comment
Open

Pre-parse JSON for proper format and all required elements #17

fuddster opened this issue Feb 12, 2022 · 1 comment

Comments

@fuddster
Copy link
Collaborator

No description provided.

@zaaj
Copy link
Contributor

zaaj commented Oct 27, 2023

No description, so not sure exactly what you have in mind for this issue.
I have noticed that the form definition JSON data is in a string, and is parsed by JavaScript in the web page, showing any parse errors on the web page.
I was thinking of removing the quotes around the JSON data, and just defining a variable to the bare JSON, changing the extension to .js and including it with a script src= tag.

This means no more friendly error messages on the web page, but buys you syntax highlighting and other editor aids VS Code can offer for JavaScript coders when editing the config data.

I might consider that pre-parsing JSON, by the editor at least. Not sure about checking for required elements though. Probably need a separate script for that.

lstone24 added a commit to lstone24/ScoutingPASS that referenced this issue Feb 27, 2024
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

No branches or pull requests

2 participants