-
Notifications
You must be signed in to change notification settings - Fork 1
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
Create a HTML5 version of the submitter #9
Labels
Comments
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
All files were generated using Yeoman (http://yeoman.io/). First time using this so I might have forgotten to ignore some files. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Not sure why this is missing from the yeoman init command. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Also ignore the local node_modules folder. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Jade is a simple templating library (http://jade-lang.com/) that is less verbose than HTML. * Modified the build process to convert .jade files to html * Ignored compiled html files in .gitignore * Don't copy .jade files to the output folders See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Not really sure if this would be beneficial (since editing will be done through Vim mostly for this part) but it is best to be consistent. All projects now have corresponding IntelliJ modules. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Not sure why Yeoman decided to use an older version. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
NOT TESTED so things might break. But there is a significant chunk of work here that needs to be put in first. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
The grunt file was just not processing the .scss files properly. Instead of wasting time fiddling with this, I decided to just use straight CSS. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
Bootstrap's icons need to be in a /img/ folder by default. Since I don't want to touch the default CSS, I renamed the image folder instead. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 1, 2013
AngularJS relies on some naming conventions to do its dependency injection. If we use minify, it wantonly renames stuff without caring for those conventions causing all things to break. The solution is to *not* minify for now. For more information https://groups.google.com/d/msg/yeoman-dev/642ocaWEHw8/Q2bN0UTw6sgJ http://docs.angularjs.org/guide/dev_guide.services.injecting_controllers See #9.
Just found out that Coursera doesn't support Cross-Origin Resource Sharing so it will not be possible to do this just with Javascript. We would need to proxy the requests through some local server which seems way more complicated than necessary. |
vazexqi
pushed a commit
that referenced
this issue
Feb 2, 2013
To circumvent the same origin policy, I have created a simple reverse proxy using node-http-proxy (https://github.com/nodejitsu/node-http-proxy). To initialize the entire system, be sure to call `npm install` followed by `npm start`. See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 2, 2013
1) Ensure that the submission factory *must* be returned. 2) Fix typo for function name respondToChallenge(). See #9.
vazexqi
pushed a commit
that referenced
this issue
Feb 2, 2013
This makes use of our reverse proxy to circumvent the same origin policy for XMLHTTPRequest(). Code is in a bit of a mess since I don't know of a good way to properly organize the callbacks. See #9.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Instead of having to make the students install Java, would it be possible to have the submitter be a web browser app that uses the following technology?
The text was updated successfully, but these errors were encountered: