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

bug: no way to identify why app installation failed #1030

Closed
4 tasks done
ILoveOpenSourceApplications opened this issue Jul 15, 2023 · 6 comments · Fixed by #1528 or #1573
Closed
4 tasks done

bug: no way to identify why app installation failed #1030

ILoveOpenSourceApplications opened this issue Jul 15, 2023 · 6 comments · Fixed by #1528 or #1573
Assignees
Labels
Bug report Something isn't working

Comments

@ILoveOpenSourceApplications

Type

Error at runtime

Bug description

There will be cases where one may be forced to uninstall the manager due to some issue and the user could have forgotten to backup the keystore. In such cases when updating a patched app over an existing patched app will end app with the message "app not installed as package conflicts with an existing package". But after the recent update and l presume with the fix #1021, when clicking install the pop up after installation just disappears. If installed successfully, the manager will say installed. But if it fails there is no way to know what's the issue and manager just says installing.

Steps to reproduce

  1. Some how create a situation where the manager will come across a conflict (like mentioned above).
  2. Try to install the same app over the existing one.
  3. Normally there will the be conflict message but now you just sit there in the manager with unlimited install clicks.

Android version

Android 13

Manager version

1.4.1

Target package name

Any

Target package version.

Any

Installation type

Non-root

Patches selected.

Nil

Device logs (exported using Manager settings).

Nil

Installer logs (exported using Installer menu option) [unneeded if the issue is not during patching].

No response

Screenshots or video

screen-20230716-010252.2.mp4

Solution

No response

Additional context

No response

Acknowledgments

  • I have searched the existing issues; this is new and no duplicate or related to another open issue.
  • I have written a short but informative title.
  • I properly filled out all of the requested information in this issue.
  • The issue is solely related to ReVanced Manager and not caused by patches.
@ILoveOpenSourceApplications ILoveOpenSourceApplications added the Bug report Something isn't working label Jul 15, 2023
@ILoveOpenSourceApplications ILoveOpenSourceApplications changed the title bug: no way to identify if app installation failed bug: no way to identify why app installation failed Jul 15, 2023
@TheAabedKhan
Copy link
Member

TheAabedKhan commented Jul 15, 2023

I am currently working on the fix. Can you please turn off the Google Play Protect for some times and try to install the app again and share me the results please?

@ILoveOpenSourceApplications
Copy link
Author

ILoveOpenSourceApplications commented Jul 16, 2023

I don't think it's due to play protect. But I did what you requested and turned of play protect and repeated the same with the results also being the same. Previously the manager would show a confirmation dialog which is part of the stock package installer. Now that it's removed there's no way of knowing what happened.

The package dialog I meant:
Screenshot_20230716-103830~2

And similarly one the installation/update of an app is done that confirmation is also gone as well. Which is fine but feels odd as the users used to see that confirmation in the previous versions.

Edit: The confirmation dialog I meant which is still there:
Screenshot_20230716-131447~2

The confirmation dialog I meant which is no longer there in manager:
Screenshot_20230716-131449~2

@sssh7710
Copy link

sssh7710 commented Aug 4, 2023

The same happens to me when I install revanced

@sssh7710
Copy link

sssh7710 commented Aug 4, 2023

I'm uncomfortable because I don't know if the installation is in progress

@Ushie
Copy link
Member

Ushie commented Sep 2, 2023

I think the callback should be removed if it's not possible to pop the dialog ONLY when it errors

The dialog needs to stay for two reasons:

  • it natively contains an "Open" button to open the installed app
  • currently, many installations fail because the dialog doesn't stay up long enough to show the play protect dialog

@KobeW50
Copy link
Contributor

KobeW50 commented Oct 5, 2023

On a related note, the package installer dialog also does not appear when updating the Manager from within the Manager.

Movie001.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Something isn't working
Projects
None yet
6 participants