code-scanner icon indicating copy to clipboard operation
code-scanner copied to clipboard

Published 20 hours ago • verified publisher icon czlucius

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

Open czlucius opened this issue 3 years ago • 9 comments

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

czlucius avatar Feb 17 '22 14:02 czlucius

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

PaT0191 avatar Oct 14 '22 16:10 PaT0191

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

czlucius avatar Oct 16 '22 03:10 czlucius

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: https://github.com/czlucius/code-scanner/issues/11

PaT0191 avatar Oct 16 '22 06:10 PaT0191

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

czlucius avatar Oct 16 '22 08:10 czlucius

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

PaT0191 avatar Oct 16 '22 11:10 PaT0191

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

czlucius avatar Oct 16 '22 14:10 czlucius

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.

czlucius avatar Oct 16 '22 14:10 czlucius

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! 🎃

czlucius avatar Oct 16 '22 14:10 czlucius

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.

PaT0191 avatar Oct 16 '22 15:10 PaT0191