Don't assign the store to controllers that haven't been instantiated. #362

Merged
merged 1 commit into from Oct 17, 2012

Conversation

Projects
None yet
3 participants
@bradleypriest
Member

bradleypriest commented Aug 14, 2012

This PR is a follow on to emberjs/ember.js#1269 and won't work until it has been merged

@travisbot

This comment has been minimized.

Show comment Hide comment
@travisbot

travisbot Aug 14, 2012

This pull request fails (merged a635f95b into 5b2b1c7).

This pull request fails (merged a635f95b into 5b2b1c7).

@wagenet

This comment has been minimized.

Show comment Hide comment
@wagenet

wagenet Oct 17, 2012

Owner

@bradleypriest this doesn't merge cleanly. Can you take a look?

Owner

wagenet commented Oct 17, 2012

@bradleypriest this doesn't merge cleanly. Can you take a look?

@bradleypriest

This comment has been minimized.

Show comment Hide comment
@bradleypriest

bradleypriest Oct 17, 2012

Member

@wagenet done

Member

bradleypriest commented Oct 17, 2012

@wagenet done

wagenet added a commit that referenced this pull request Oct 17, 2012

Merge pull request #362 from bradleypriest/controller-instantiation
Don't assign the store to controllers that haven't been instantiated.

@wagenet wagenet merged commit d937c05 into emberjs:master Oct 17, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment