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

Integrate build configs into flavors #642

Merged
merged 17 commits into from
Jan 9, 2023

Conversation

maxammann
Copy link
Member

@maxammann maxammann commented Nov 21, 2022

Select the build config based on a flavor instead of a gradle project property. Not sure if this is the way to go.

We are also hitting: pikaju/flutter-maps-launcher#27

If builds succeed then this is ready for review

  • You need to install latest IntelliJ / Android Studio
  • Test this by downloading the artefacts (APKs) for Nuernberg, Bayern and BayernFloss from the CI

@maxammann maxammann marked this pull request as draft November 21, 2022 14:58
@maxammann maxammann marked this pull request as ready for review December 19, 2022 19:39
@maxammann maxammann requested review from michael-markl, sarahsporck and f1sh1918 and removed request for michael-markl December 19, 2022 19:39
Copy link
Member

@michael-markl michael-markl left a comment

Choose a reason for hiding this comment

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

Everything seems to be working for me.
I tested the release builds for bayern and nuernberg, did not test bayern-floss.

.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
frontend/android/app/build.gradle Outdated Show resolved Hide resolved
frontend/android/app/build.gradle Show resolved Hide resolved
frontend/android/app/buildConfigs.gradle Show resolved Hide resolved
@@ -16,7 +16,7 @@
<option name="taskNames">
<list>
<option value="run" />
<option value="--args=&quot;execute&quot;" />
<option value="--args=&quot;create-admin bayern.ehrenamtskarte.app REGION_MANAGER [email protected] Administrator! 1&quot;" />
</list>
</option>
<option name="vmOptions" value="" />
Copy link
Member

Choose a reason for hiding this comment

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

Please readd the Run backend run config

Copy link
Member Author

Choose a reason for hiding this comment

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

Forgot to push something, fixed

Copy link
Member

Choose a reason for hiding this comment

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

Are you sure you fixed it? I can't see it in the commits.

Co-authored-by: Michael Markl <[email protected]>
@maxammann maxammann merged commit 13d8750 into whitelabel Jan 9, 2023
@maxammann maxammann deleted the integrate-build-configs-flavors branch January 9, 2023 09:16
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