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

Fix error throw when router view change if tabset directive is included #180

Closed
wants to merge 4 commits into from

Conversation

lanocturne
Copy link
Contributor

When tabset's view is destroyed, for example, the router change. There's an EXCEPTION: Attempt to use a dehydrated detector. throw on ng2-beta.6.

Reference Issue: @angular/angular#6786

When tabset's view is destroyed, for example, the router change. There's an ```EXCEPTION: Attempt to use a dehydrated detector.``` throw on ng2-beta.6.
@valorkin valorkin changed the title Fix error throw when router view change if tabset directive is included Fix error throw when router view change if tabset directive is included Feb 12, 2016
@valorkin
Copy link
Member

Hey, looks good
Can you please squash commits to one and force push to your branch
and apply angular commit naming
commit name: fix(tabs): ignore destroyed tabs...
thing is: this should not happen if router will be destroying view content correctly

@lanocturne
Copy link
Contributor Author

Definitely.

Dmitriy Shekhovtsov [email protected]於 2016年2月12日 週五,下午10:20寫道:

Hey, looks good
Can you please squash commits to one and force push to your branch
and apply angular commit naming
commit name: fix(tabs): ignore destroyed tabs...
thing is: this should not happen if router will be destroying view content
correctly


Reply to this email directly or view it on GitHub
#180 (comment)
.

Thanks,
Stanislaus

@binarious
Copy link

👍

@brian428
Copy link

Any thoughts on when this will be merged?

@valorkin
Copy link
Member

Right now I am updating ng2-bootstrap up to beta7
so today will be merging testing all PRS
#214

@valorkin valorkin closed this in ae8c617 Feb 24, 2016
@binarious
Copy link

Still getting errors like Error: Attempt to use a dehydrated detector: DeliveryShow_2 -> select on leaving a component with tabs. (ng2 beta7 + n2-bootstrap v1.0.4)

@binarious
Copy link

Well the fix landed in v1.0.5. Now it works.

@lanocturne
Copy link
Contributor Author

Glad to hear that.

binarious [email protected]於 2016年2月25日 週四,下午12:25寫道:

Well the fix landed in v1.0.5. Now it works.


Reply to this email directly or view it on GitHub
#180 (comment)
.

Thanks,
Stanislaus

@valorkin
Copy link
Member

@lanocturne I have squashed commits, so you can see release notes now :)
https://github.com/valor-software/ng2-bootstrap/releases

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.

5 participants