Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(SplashScreen): show method not resolving if autoHide is false #3144

Merged
merged 3 commits into from
Jun 29, 2020

Conversation

sergiomilici
Copy link
Contributor

When using the plugin in TS, the call Splash.show({autoHide:false}) was never resolving the returned promise because the native call to complete the call was never invoked. It was only called with autoHide = true.

fixes: #3143

@sergiomilici sergiomilici changed the title Fix/splash screen show Fix: splash screen show method not resolving returned promise Jun 23, 2020
@sergiomilici sergiomilici changed the title Fix: splash screen show method not resolving returned promise fix: splash screen show method not resolving returned promise Jun 23, 2020
@jcesarmobile jcesarmobile changed the title fix: splash screen show method not resolving returned promise fix(Splash Screen): show method not resolving if autoHide is false Jun 29, 2020
@jcesarmobile jcesarmobile changed the title fix(Splash Screen): show method not resolving if autoHide is false fix(SplashScreen): show method not resolving if autoHide is false Jun 29, 2020
Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

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

good catch, thanks!

@jcesarmobile jcesarmobile merged commit 28a0e42 into ionic-team:master Jun 29, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: SplashScreen: Splash.show promise never resolves when SplashScreenShowOptions are provided
2 participants