-
Notifications
You must be signed in to change notification settings - Fork 429
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
AAB enabled in the app #424
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the scripts and not the bundle configuration. Use badge magic for reference
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CloudyPadmal Please make sure that fastlane is initialised in this repository
bc77e0c
to
9358462
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll add fastlane in a later stage. Just build the appbundle for now.
@CloudyPadmal he updated the pr with the required files. That's all we need as of now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only ./gradlew bundleRelease
is needed in this PR and modify upload-apk.sh
script to upload that file to apk branch and add it to git lfs.
Fixed in #428 |
Fixes #401
Changes: AAB enabled in the app
Screenshot/s for the changes:
N/A
Checklist: [Please tick following check boxes with
[x]
if the respective task is completed]strings.xml
,dimens.xml
andcolors.xml
without hard-coding themstrings.xml
,dimens.xml
orcolors.xml
APK for testing:
N/A