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

Default to .aab builds with Play Store #48

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

arno-le
Copy link

@arno-le arno-le commented Oct 17, 2019

Changed template Fastfile to use Android App Bundle as default.
Reasons to use .aab include smaller app size and no need to build for separate CPU architectures, as that is handled by the Play Store.
More info here:
https://developer.android.com/platform/technology/app-bundle

As of now, Crashlytics does not support .aab -format.

@plrdev
Copy link
Contributor

plrdev commented Dec 6, 2019

Hey thanks for the PR. This is definitely something we want to support, but probably would be good to make it a user decision, which option to choose from. It would require adding a prompt and changing the template depending on the choice. Would you be willing to add that?

@arno-le
Copy link
Author

arno-le commented Dec 18, 2019

Sure!

@arno-le
Copy link
Author

arno-le commented Jan 13, 2020

@eduplus Added a prompt to select the build variant. Additionally, I feel like the askQuestion method in android.js should be refactored or removed

@plrdev
Copy link
Contributor

plrdev commented Jan 23, 2020

Thanks, this looks good to me, @miguelrrc promised to check that it works with his (long in waiting) PR, which does a lot on Android side as well. Hopefully we can merge these both in soon.

@miguelrrc
Copy link
Collaborator

It is good to go! 👍

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.

4 participants