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

[FR] Stop Requesting Enabling Auto-Resolution for Android Resolver When Already Denied #689

Open
SCastanedaMunoz opened this issue Jun 13, 2024 · 8 comments
Labels

Comments

@SCastanedaMunoz
Copy link

[READ] Guidelines

When filing a feature request please make sure the issue title starts with "FR:".

A good feature request ideally

  • is either immediately obvious (i.e. "Add Sign in with Apple support"), or
  • starts with a use case that is not achievable with the existing Firebase API and
    includes an API proposal that would make the use case possible. The proposed API
    change does not need to be very specific.

Once you've read this section, please delete it and fill out the rest of the template.

Feature proposal

  • EDM4U Component: Android Resolver(Android Resolver, iOS Resolver, Version Handler, Package Manager, etc)

We currently disable auto-resolution for the EDM since it overrides any existing private maven repositories or custom modifications to gradle files. However, when disabling the auto-resolution of dependencies, a popup appears every time we compile/modify code. The popup ask to enable auto resolution, when denying this, nothing happens and the popup appears on the next compilation.

Enabling auto-resolution and disabling it seems to make the pop-up disappear for a while but it does not last long.

@SCastanedaMunoz
Copy link
Author

Screenshot 2024-06-13 at 4 34 29 PM

This is the pop-up in question, pressing "Disable" effectively does nothing. Then the pop-up just keeps spamming on every code change. 👎🏻

@argzdev argzdev removed the new to be triaged label Jun 14, 2024
@SCastanedaMunoz
Copy link
Author

Any updates on this? The EDM keeps enabling auto-resolution by itself, which is very annoying and blocks development a lot.

@argzdev
Copy link
Collaborator

argzdev commented Jun 20, 2024

Hey @SCastanedaMunoz, thanks for reaching out. This sounds like a bug, however after disabling auto-resolution, I'm not encountering this behavior when I do any codes with our Unity quickstart. I wonder if there are extra steps to exhibit this behavior. Any chance you could provide a minimal reproducible example so we can speed up the investigation? Thanks!

@argzdev argzdev added the needs-info Need information for the developer label Jun 20, 2024
@SCastanedaMunoz
Copy link
Author

Taking a look at the GvhProjectSettings.xml I can see that pressing Disable on the dialog does not actually change the flag GooglePlayServices.AutoResolutionDisabledWarning, so it remains true. I have changed this flag manually to False for the time being. I am trying multiple scenarios to see if any actions cause this flag to flip back to True

@google-oss-bot google-oss-bot added needs-attention Need Googler's attention and removed needs-info Need information for the developer labels Jun 20, 2024
@SCastanedaMunoz
Copy link
Author

Additional info: I rant some of my Unit tests from my project and most of the settings from GvhProjectSettings.xml got wiped. I think this explains why the popup keeps appearing, something seems to be causing the GvhProjectSettings.xml to get reset. This means that every time I build/run unit tests on my devices, the settings get wiped and have to disable the auto-resolution every time.

@SCastanedaMunoz
Copy link
Author

Any updates on this? 👀

@a-maurice
Copy link
Collaborator

For the GvhProjectSettings.xml issue, that should hopefully be resolved with the latest release, 1.2.180, which fixed a bug that was causing that file to be overwritten sometimes.
For the GooglePlayServices.AutoResolutionDisabledWarning not being set in code, it isn't meant to, as all that does it toggle the warning that appears in the log, and is meant to require manual disabling in the settings.

Looking at the logic for the original pop up, it does seem to be setting the correct fields if Disable is selected:

SettingsDialogObj.PromptBeforeAutoResolution = false;

My hope is that maybe the GvhProjectSettings.xml issue of it being overwritten was causing your problem, as similar to @argzdev, I haven't been able to reproduce it prompting multiple times.

@SCastanedaMunoz
Copy link
Author

Thanks, we will take a look at the 1.2.180 release and see if this fixes our issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants