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

configure: applicationId in build.gradle stays unchanged #24

Closed
tobiasmuecksch opened this issue Nov 30, 2021 · 8 comments
Closed

configure: applicationId in build.gradle stays unchanged #24

tobiasmuecksch opened this issue Nov 30, 2021 · 8 comments
Labels
enhancement New feature or request

Comments

@tobiasmuecksch
Copy link

Current behavior

When running cap-configure it seems to ignore the applicationId for android

plaftorm-config.yml

vars:
  PACKAGE_NAME_ANDROID:
    default: com.capconfigure.demoandroid

  android:
    packageName: $PACKAGE_NAME_ANDROID
    versionName: 2.2.2
    versionCode: 222

Bildschirmfoto 2021-11-30 um 07 23 02

Version

I observed this behavior in version 1.0.20

Reproduction

I've created a demo repo for you to quickly reproduce.

https://github.com/tobiasmuecksch/cap-configure-problem-demo

  1. clone repo
  2. run npm run init
    • Info: This task installs npm packages, adds the ios and android platforms and runs cap-configure

Then check the android/app/build.gradle-file, and you'll see, that the applicationId is not updated correctly.

@mlynch
Copy link
Contributor

mlynch commented Nov 30, 2021

Okay I need to add that as it's currently only change the package name in the AndroidManifest.xml

@mlynch mlynch added the enhancement New feature or request label Dec 5, 2021
@ksteuer
Copy link
Contributor

ksteuer commented Dec 9, 2021

Hi @mlynch. Do you think we could also implement changing the package name in MainActivity.java (and other classes if exist) and its parent directory according to provided bundleId?

@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2021

Yes, definitely. Perhaps I'll roll these two into a single task

@ksteuer
Copy link
Contributor

ksteuer commented Dec 9, 2021

The first one should be done in my recent PR. I will try to take care of the second one in the following days.

@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2021

Awesome! Left a comment on the PR. I wasn't expecting a PR so I had already started on this and the package folder rename. If I get it wrapped up here shortly I'll let you know otherwise you can go ahead

@ksteuer
Copy link
Contributor

ksteuer commented Dec 9, 2021

OK! I will have more time tomorrow evening and over the weekend. If you had already started I can take care of another issue though.

@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2021

I'll throw up my first pass as a PR and would like your input!

@mlynch
Copy link
Contributor

mlynch commented Dec 9, 2021

@ksteuer #35 would love your feedback/testing/input!

mlynch added a commit that referenced this issue Dec 10, 2021
First pass at Android package rename and file modification. #24
@mlynch mlynch closed this as completed Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants