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

Implement RFC 57 - Reference Unification #3303

Merged
merged 1 commit into from
Dec 15, 2015
Merged

Implement RFC 57 - Reference Unification #3303

merged 1 commit into from
Dec 15, 2015

Conversation

pangratz
Copy link
Member

@@ -0,0 +1,77 @@
import Ember from 'ember';
// FIXME doesn't work an Model is undefined
// import Model from '../model';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably due to circular dependencies which are not supported right now

@pangratz pangratz changed the title Implement RFC 57 - Reference Unification [WIP] Implement RFC 57 - Reference Unification Jun 22, 2015
@jcope2013
Copy link
Contributor

status on this?

@pangratz pangratz changed the title [WIP] Implement RFC 57 - Reference Unification Implement RFC 57 - Reference Unification Oct 6, 2015
@bmac
Copy link
Member

bmac commented Dec 11, 2015

@pangratz do you mind rebasing this pr and adding feature flags so this can be merged?

@pangratz
Copy link
Member Author

@bmac done...

bmac added a commit that referenced this pull request Dec 15, 2015
Implement RFC 57 - Reference Unification
@bmac bmac merged commit 5826230 into emberjs:master Dec 15, 2015
@bmac
Copy link
Member

bmac commented Dec 15, 2015

🎉

@sandstrom
Copy link
Contributor

🎈 🎆 🌟

I thought this was still pending approval from the core team (RFC-thread). Then suddenly, as gift from the heavens, it's landing in master.

@workmanw
Copy link

Wow. I was just catching up on some things and came across this (and the RFC). 👍. This will definitely help us out in our app. Thank you!

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.

6 participants