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

Remove deprecated Android SplashScreenDrawable. #539

Merged
merged 3 commits into from
Aug 17, 2023

Conversation

Jonas-Sander
Copy link
Collaborator

@Jonas-Sander Jonas-Sander commented Mar 31, 2023

We still used an deprecated way to declare a splash screen, which caused the following warning to be printed on Android start-up: W/FlutterActivityAndFragmentDelegate( 6179): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps.

Since we also already use the recommended way to declare a splash screen additionally, we just had to remove the old way. There are no behavior / splash screen changes, everything works as it used to.

We still used an deprecated way to declare a splash screen, which caused the following warning to be printed on Android start-up: `W/FlutterActivityAndFragmentDelegate( 6179): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps.`

Since we also already use the recommand way to declare a splash screen additionally, we just had to remove the old way. There are no behavior / splash screen changes, everything works as it used to.
@docs-page
Copy link

docs-page bot commented Mar 31, 2023

To view this pull requests documentation preview, visit the following URL:

docs.page/sharezoneapp/sharezone-app~539

Documentation is deployed and generated using docs.page.

@github-actions
Copy link

github-actions bot commented Mar 31, 2023

Visit the preview URL for this PR (updated for commit d347bac):

https://sharezone-test--pr539-remove-deprecated-an-3z8847fl.web.app

(expires Thu, 17 Aug 2023 00:26:04 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b

@nilsreichardt nilsreichardt added the build-app-preview A label that can be assigned to a PR to trigger a build of the app preview. label Apr 1, 2023
@Sharezone-Admin
Copy link
Member

Sharezone-Admin commented Apr 1, 2023

⬇️ Generated builds by Codemagic for commit d347bac ⬇️

Note: Only Sharezone team members are able to install the iOS app.

iOS Android
image
Download link
image
Download link

Copy link
Member

@nilsreichardt nilsreichardt left a comment

Choose a reason for hiding this comment

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

LGTM

@nilsreichardt nilsreichardt added this pull request to the merge queue Aug 9, 2023
@nilsreichardt nilsreichardt removed this pull request from the merge queue due to a manual request Aug 9, 2023
@nilsreichardt nilsreichardt removed the build-app-preview A label that can be assigned to a PR to trigger a build of the app preview. label Aug 10, 2023
@nilsreichardt nilsreichardt added this pull request to the merge queue Aug 10, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 10, 2023
@nilsreichardt nilsreichardt added this pull request to the merge queue Aug 17, 2023
Merged via the queue into main with commit d566510 Aug 17, 2023
@nilsreichardt nilsreichardt deleted the remove-deprecated-android-splash-screen branch August 17, 2023 23:00
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.

3 participants