Skip to content

[No QA] Make BootSplash have better parity on web/desktop#2623

Merged
francoisl merged 1 commit into
mainfrom
Rory-BootsplashWebParity
Apr 29, 2021
Merged

[No QA] Make BootSplash have better parity on web/desktop#2623
francoisl merged 1 commit into
mainfrom
Rory-BootsplashWebParity

Conversation

@roryabraham

Copy link
Copy Markdown
Contributor

Details

react-native-bootsplash returns a void promise for hide, show, and a promise that resolves to one of "visible", "hidden", "transitioning" for getVisibilityStatus. This PR just improves the parity in our web shim so that we don't have another problem like the one solved by this PR

Fixed Issues

n/a

Tests

Just do some basic regression testing to make sure the app still runs okay and doesn't have any console warnings.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

@roryabraham roryabraham self-assigned this Apr 29, 2021
@roryabraham roryabraham requested a review from a team as a code owner April 29, 2021 06:35
@MelvinBot MelvinBot requested review from francoisl and removed request for a team April 29, 2021 06:35
@francoisl francoisl merged commit 084c376 into main Apr 29, 2021
@francoisl francoisl deleted the Rory-BootsplashWebParity branch April 29, 2021 18:49
@OSBotify

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@roryabraham roryabraham changed the title Make BootSplash have better parity on web/desktop [No QA] Make BootSplash have better parity on web/desktop Apr 29, 2021
@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging in version: 1.0.34-1🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify

OSBotify commented May 8, 2021

Copy link
Copy Markdown
Contributor

🚀 Deployed to production in version: 1.0.39-5🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

3 participants