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

[BUGFIX beta] QP routes named after Obj.prototype props should work #11878

Merged
merged 1 commit into from
Jul 24, 2015

Conversation

machty
Copy link
Contributor

@machty machty commented Jul 24, 2015

Fixes #11877

@stefanpenner
Copy link
Member

We may also want to do a 1.13.x release of this.

@machty
Copy link
Contributor Author

machty commented Jul 24, 2015

Wow. Feels like it's been years since I got green on my first try.

@rwjblue
Copy link
Member

rwjblue commented Jul 24, 2015

The implementation would be slightly different if we wanted this in release (can't use Object.create), but this doesn't seem to fit the bill of a BUFIX release to me. It isn't a massive bug affecting many users (one reported case since 1.7). I'd rather try to keep to our 'normal' process here.

rwjblue added a commit that referenced this pull request Jul 24, 2015
[BUGFIX beta] QP routes named after Obj.prototype props should work
@rwjblue rwjblue merged commit 82b123c into emberjs:master Jul 24, 2015
@rwjblue rwjblue deleted the qp-object-prototype-clash-fix branch July 24, 2015 04:04
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