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

Backbone-on-Rails Gem #11

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

Conversation

mulderp
Copy link

@mulderp mulderp commented Jul 23, 2012

Hi Addy,

first, thanks for your Backbone-boilerplate examples and your writings on BackboneJS. I am learning a lot from these.

I have adapted your Rails example, which uses the Gem Backbone-Rails to an example that uses the Gem Backbone-on-Rails by @meleyal
The main diference is the setup of the Application view, and a slight modification of the client-side MVC file structure.
I was playing with this in the context of learning more on Rails-based client-side MVC, and maybe it's helpful for others.
Thanks for your feedback.

Cheers,
Patrick

@addyosmani
Copy link
Owner

Thanks for this @mulderp! (and my apologies for the delay in taking a look at your pull request).

Do you think it makes sense to create a Rails sub-section in the readme for the setups we provide which involve Rails? We seem to be building up a few variations at the moment.

@mulderp
Copy link
Author

mulderp commented Aug 22, 2012

Sounds like a good idea.
Additionally, we might think in using separate repositories. That would have the advantage to get some numbers who is following what backend-stack, and how much change there are in commits / interest.

The server stacks that I read about in books or user groups are JS/Node - Ruby/Rails/Sinatra - PHP - Java/Grails - Python

@mulderp
Copy link
Author

mulderp commented Aug 22, 2012

But I will think about some small text in the README for the Rails setup so far. Update in the next days hopefully.

@meleyal
Copy link

meleyal commented Mar 1, 2017

5 years late to the party but just stumbled on this in my PR queue. @addyosmani can you merge or close this, thanks!

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

Successfully merging this pull request may close these issues.

3 participants