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

It should not require a JavaScript runtime #28

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

lucassus
Copy link
Owner

  • find a good solution for seperating js from ruby
  • create simple angularjs app with grunt/requirejs and friends

separate front end from the backend (sinatra app)

  • move all front-end related files to the separate folder
  • get rid of sprockets
  • create grunt build file
  • fix javascript unit tests
  • bower for underscore and bootbox
  • setup proxy for bower
  • fix bootbox dialogs
  • fix angularjs e2e test
  • build production release to ./public directory
  • minify css production files
  • fix images in production release
  • figure out how to run e2e specs
  • update build script, run all specs on travis-ci
  • rename lib to backend
  • do not include all files in the gem
  • update script for builiding the gem
  • update Readme
  • create Readmne for the front-end application
  • get rid of Sinatra?

@rusdec
Copy link

rusdec commented Jan 15, 2013

add gem 'execjs' and gem 'therubyracer' into Gemfile

@lucassus
Copy link
Owner Author

thx. @rusdec I'll check it!

@ghost ghost assigned lucassus Jan 15, 2013
lucassus added a commit that referenced this pull request Jul 14, 2013
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 262865e on refactor-split-the-app into c350969 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling 61f30ae on refactor-split-the-app into c350969 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling 1e7bd32 on refactor-split-the-app into c350969 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants