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

Bug: Not Compatible with 2.10 (Glimmer 2) #50

Merged

Conversation

AlexisMontagne
Copy link
Contributor

@AlexisMontagne AlexisMontagne commented Apr 18, 2017

This project uses semver, please check the scope of this pr:

  • #patch# - backwards-compatible bug fix
  • #minor# - adding functionality in a backwards-compatible manner
  • #major# - incompatible API change

CHANGELOG

Delay the initialization of the block-slot to the didInsertElement callback.

Assertion Failed: You modified "isActive" twice on <xxx@component:yield-slot::ember1525> in a single render. It was rendered in "component:yield-slot" and modified in "component:block-slot". This was unreliable and slow in Ember 1.x and is no longer supported. See https://github.com/emberjs/ember.js/issues/13948 for more details.


This change is Reviewable

@coveralls
Copy link

coveralls commented Apr 18, 2017

Coverage Status

Changes Unknown when pulling f4b8b85 on upfluence:am-fix-glimmer-2 into ** on ciena-blueplanet:master**.

@sglanzer-deprecated
Copy link
Contributor

Sorry about the delay on this - literally just got my email backlog down to the point where I saw it!

@sglanzer-deprecated
Copy link
Contributor

sglanzer-deprecated commented Apr 26, 2017

Approved

Approved with PullApprove

@sglanzer-deprecated
Copy link
Contributor

sglanzer-deprecated commented Apr 26, 2017

Approved

Approved with PullApprove

@sglanzer-deprecated sglanzer-deprecated merged commit ba6e004 into ciena-blueplanet:master Apr 26, 2017
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.

None yet

3 participants