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

Firebase: Migrate to using the Kotlin extensions (KTX) APIs in the main modules #1062

Merged
merged 11 commits into from
Jul 8, 2024

Conversation

SimonMarquis
Copy link
Contributor

@SimonMarquis SimonMarquis commented Nov 29, 2023

https://firebase.google.com/docs/android/kotlin-migration

Requirements: Firebase BoM 32.5.0, but let's update to the latest 32.6.0.

And remove warnings:

  • 'logEvent(String, ParametersBuilder.() -> Unit): Unit' is deprecated. Migrate to use the KTX API from the main module: https://firebase.google.com/docs/android/kotlin-migration.
  • 'param(String, String): Unit' is deprecated. Migrate to use the KTX API from the main module: https://firebase.google.com/docs/android/kotlin-migration.

See #1006 for these new permissions and "Ad Services" property.

…in modules

https://firebase.google.com/docs/android/kotlin-migration

Requirements: Firebase BoM 32.5.0, but let's update to the latest 32.6.0.

And remove warnings:
- `'logEvent(String, ParametersBuilder.() -> Unit): Unit' is deprecated. Migrate to use the KTX API from the main module: https://firebase.google.com/docs/android/kotlin-migration.`
- `'param(String, String): Unit' is deprecated. Migrate to use the KTX API from the main module: https://firebase.google.com/docs/android/kotlin-migration.`
See android#1006 for these new permissions and "Ad Services" property.
@SimonMarquis
Copy link
Contributor Author

@dturner this PR is ready to be reviewed 🙇

@dturner dturner merged commit 7d04af0 into android:main Jul 8, 2024
4 checks passed
@SimonMarquis SimonMarquis deleted the firebase-ktx branch July 8, 2024 18:34
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.

2 participants