[Bug]: Android CI will crash due to undeclared variables (in .gitignore)

Question

[Bug]: Android CI will crash due to undeclared variables (in .gitignore)

czlucius opened this issue 3 years ago · 9 comments

As some variables aren't pushed up, Android CI will fail (for future commits)

Answer 1 · 2022-10-14T16:11:34.000Z

Hello, I would like to work on this issue, can you assign this to me?

Answer 2 · 2022-10-16T03:50:18.000Z

Sorry for the delay in response.
Do you have a plan on how you're going to approach this?
There are some API secrets that are definitely not going to be pushed up, and this causes compilation to fail for the play build flavor.
Apart from that there may be other issues.
Do let me know, and you can create a PR if you want as well.
@PaT0191

Answer 3 · 2022-10-16T06:58:18.000Z

Hi, no problem.
Hmmm, i can try to fix it anyway and if i wouldnt be able to fix it, i can work on this one: #11

Answer 4 · 2022-10-16T08:28:26.000Z

Can you make a PR first or tell me what you intend to do?

Answer 5 · 2022-10-16T11:26:28.000Z

Using new MaterialAlertDialogBuilder.Builder(context) instead of new MaterialAlertDialogBuilder(context) in AddWifiAction.java:69, cant be the problem?

Answer 6 · 2022-10-16T14:33:48.000Z

The problem is not solely caused by that (although it is, as you correctly pointed out, an error)
The main problem as addressed in the title is because the play variant uses a secret key that cannot be pushed up but is required for compilation.

Due to low usage, I'm thinking of removing ads. Could you help to remove advertising (AdStrategy2) in all variants?
Thanks

Answer 7 · 2022-10-16T14:37:03.000Z

Using new MaterialAlertDialogBuilder.Builder(context) instead of new MaterialAlertDialogBuilder(context) in AddWifiAction.java:69, cant be the problem?

Thanks for spotting that bug. It was committed recently in a PR.

Answer 8 · 2022-10-16T14:41:57.000Z

If you'd like, can you submit a PR removing AdStrategy2 and fixing the MaterialAlertDialogBuilder code? (and test if it can build locally on your machine)
Can remove strings and string references to idappad and idbannerad
It would be much appreciated.

After you open a PR, I will assign this to you.

Thanks and happy hacking/contributing! 🎃

Answer 9 · 2022-10-16T15:07:20.000Z

The problem is not solely caused by that (although it is, as you correctly pointed out, an error) The main problem as addressed in the title is because the play variant uses a secret key that cannot be pushed up but is required for compilation.

Due to low usage, I'm thinking of removing ads. Could you help to remove advertising (AdStrategy2) in all variants? Thanks

Yeap, i can help you with that.

Thanks for spotting that bug. It was committed recently in a PR.

If you'd like, can you submit a PR removing AdStrategy2 and fixing the MaterialAlertDialogBuilder code? (and test if it can build locally on your machine) Can remove strings and string references to idappad and idbannerad It would be much appreciated.

After you open a PR, I will assign this to you.

Thanks and happy hacking/contributing! 🎃

Ill get on computer tomorow soo, then ill create PR for that.
Thank you.