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

bail if no document so you can require on server #91

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

Conversation

blented
Copy link

@blented blented commented Jul 3, 2013

useful for shared server / client code like backbone views

@madrobby
Copy link
Owner

madrobby commented Jul 3, 2013

What's your use case to require it server-side?

@blented
Copy link
Author

blented commented Jul 4, 2013

Shared Backbone views rendered on both client and server. The first page
load is rendered on the server, subsequent navigation is rendered client
side. The implementation is somewhat similar to
Rendrhttps://github.com/airbnb/rendr
.

Grant
On Jul 3, 2013 1:55 PM, "Thomas Fuchs" notifications@github.com wrote:

What's your use case to require it server-side?


Reply to this email directly or view it on GitHubhttps://github.com//pull/91#issuecomment-20444471
.

useful for shared server / client code like backbone views
- if key doesn't have mod, key isn't set
- should continue, not return as there are more keys to check
- I get a "key is not defined" error when requiring, exporting
global.key fixes it
@blented
Copy link
Author

blented commented Dec 28, 2013

Added a couple more commits, one for global.key and one for unbindKey. The comments on both commits detail the tweaks.

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.

2 participants