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

Commit google-services.json #100

Merged
merged 3 commits into from
Apr 20, 2022
Merged

Commit google-services.json #100

merged 3 commits into from
Apr 20, 2022

Conversation

martinbonnin
Copy link
Contributor

@martinbonnin martinbonnin commented Apr 20, 2022

Make it easier to contribute to the project by adding the firebase API keys. This is ok because the keys are in the APK anyways so there's no absolute way to secure them. They are just a bit more visible now. Also the keys are restricted for the fr.paug.androidmakers packageIds and the debug/release SHA1s.

Because I use the namespace API from the AGP, the project now requires Android Studio Dolphin to be able to start the activities from the IDE

)
)
Surface(
Surface(color = MaterialTheme.colors.background) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a small unrelated UI change to make sure the time separator is opaque


project.androidSetup()

project.extensions.findByName("android")!!.apply {
this as BaseExtension
defaultConfig.apply {
applicationId = "fr.paug.androidmakers"
namespace = "fr.paug.androidmakers"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to give a shot at the namespace thing, which took me down the rabbit hole of updating to the latest AS canary because Bumblebee doesn't find the application to start if package is not in the Manifest
. It's all working with Dolphin!

@@ -1,13 +1,13 @@
[versions]
kotlin = "1.6.10"
kotlin = "1.6.10" # bumping to 1.6.20 breaks IDE autocomplete
Copy link
Contributor Author

@martinbonnin martinbonnin Apr 20, 2022

Choose a reason for hiding this comment

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

@martinbonnin martinbonnin merged commit c4790e9 into main Apr 20, 2022
@martinbonnin martinbonnin deleted the add-google-services branch April 20, 2022 13:55
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