Since AMD support, default modelScope no longer includes 'window' #459

Open
philfreo opened this Issue May 8, 2014 · 1 comment

Comments

Projects
None yet
2 participants
Collaborator

philfreo commented May 8, 2014

I've always used RequireJS, but when upgrading my app to the latest Backbone-RelationaI, which now has AMD support, some code broke because it seems that the default Backbone.Relational.store._modelScopes no longer includes window by default, in AMD settings.

I don't mind this behavior at all, but I figured you might want to either a) change this if unintentional, or b) include a note in the Change Log that a call to addModelScope might be necessary

@PaulUithol PaulUithol added the docs label May 21, 2014

Owner

PaulUithol commented May 21, 2014

Good point, I'll include it in the docs.

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