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

Prevent TypeError "Cannot read property 'getActiveChildNavs' of undefined" #323

Merged
merged 1 commit into from Aug 24, 2018

Conversation

ChrizC
Copy link
Contributor

@ChrizC ChrizC commented Jul 24, 2018

This pull request resolves a potential issue in super-tabs when trying to use multiple super-tabs instances in different pages in the same app. The plugin would break with the TypeError in the title, and a re-start would be needed to fix. This now allows the app to continue as normal.

@danielsogl danielsogl self-assigned this Jul 25, 2018
@ChrizC
Copy link
Contributor Author

ChrizC commented Jul 31, 2018

Bump - @danielsogl any plans to merge this any-time soon? It's causing issues in our app.

@danielsogl
Copy link
Contributor

I try to take a look this week! But as far as I can see it should be no problem.

@ChrizC
Copy link
Contributor Author

ChrizC commented Aug 10, 2018

@danielsogl sorry to be a pain, but bumping this for visibility

@traed
Copy link

traed commented Aug 14, 2018

Would love to see this merged as well!

@mdanics
Copy link

mdanics commented Aug 23, 2018

would also like to see this merged.

in the meantime, is there any solution?

@traed
Copy link

traed commented Aug 23, 2018

I posted a workaround in #308

@danielsogl danielsogl merged commit a0c8496 into zyra:master Aug 24, 2018
@ChrizC
Copy link
Contributor Author

ChrizC commented Aug 29, 2018

@danielsogl thanks for merging, can you release an updated version of the plugin to npm please?

@D4rkMindz
Copy link

Yeah, please.
I really like this component and i dont want to implement a dirty hack for it to work.

@danielsogl
Copy link
Contributor

cc @ihadeed

@ihadeed
Copy link
Member

ihadeed commented Sep 23, 2018

Just published ionic2-super-tabs@4.3.1

ihadeed pushed a commit that referenced this pull request Oct 23, 2018
* chore(package): bump deps and add lint rules

* WIP

* WIP

* WIP

* chore: refactor build and readme (#325)

* chore(build): add basic travis script (#326)

* fix(lib): prevent TypeError "Cannot read property 'getActiveChildNavs' of undefined" (#323)

* Remove package-lock.json

* 4.3.1

* chore(): update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants