-
Notifications
You must be signed in to change notification settings - Fork 228
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
Remove PHP requirement from the IBCM simulator #99
base: master
Are you sure you want to change the base?
Conversation
Reading is working, running...not so much
Reset and revert are now working
771b218
to
48d6111
Compare
This raises the browser requirement to: - Firefox 52 (March 2017) - Chrome 57 (March 2017) - Edge 16 (October 2017) - Safari 10.1 (March 2017)
48d6111
to
a80dfc1
Compare
Note: This was more of a fun experiment, so I won't be discouraged or anything if you don't end up merging it for Reasons, but it does work! |
I really like this, and plan to include it. But with the course just moving online, and there being less help available to the students as a result, I'm hesitant to change this right before the students are going to start on the lab. So I'll merge this in in a few weeks. Thanks for all your work on this! |
The branch name no longer makes sense, as I ended up ripping out all the PHP, haha :P.
Anyway...as JavaScript has a FileReader API, we can leverage that to do all the processing locally instead of requiring a server.
Some other changes I performed along the way:
TODO