-
Notifications
You must be signed in to change notification settings - Fork 49
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
"Expecting double quoted ..." #10
Comments
what is your browser? what precisely is the complaint? |
Yes, please give more details if you intersted in a solution. If the demo page works in your browser there is probably a syntax error in your configuration file. |
Have look at the abx, mushra, and preference config: |
I would further suggest to define the config files as JSON, instead of putting a singe JS variable in such a file. In the index.html the JS code can be easily changed.
|
Yes, that would be a little cleaner as the config does not include the
TestConfig variable definition. But that does not work anymore when running
and testing locally, right? AFAIK an ajax call to a local file is not
allowed in most browsers. This will only work when Beaqlejs is uploaded to
a Webserver.
|
I agree that this comma is not required, but I do neither get any warnings nor errors in the Firefox web console window. I have just verified this with Firefox 60.5 ESR from my Debian system. |
I expected that a web-based app would run on a remote or local web server. The things with the comma might be more a matter of coding standards. However, I was not able to run the app because of this bugs. |
can we close this then as there's no bug and possibly move the discussion about requiring a web server (which, at the same time would simplify the JS) into a different bug/feature request/issue? |
All the config files are not correct. My Browser complains about commas after the objects in the "Testset" Array and after the key-value-pairs under "Files".
The text was updated successfully, but these errors were encountered: