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

[Desktop] Sync v2 - Apps are not automatically installed #11204

Closed
btlechowski opened this issue Aug 12, 2020 · 7 comments
Closed

[Desktop] Sync v2 - Apps are not automatically installed #11204

btlechowski opened this issue Aug 12, 2020 · 7 comments

Comments

@btlechowski
Copy link

App is synced but the App is not installed

According to @darkdh this is an upstream bug

Steps to Reproduce

  1. Clean install on DeviceA and DeviceB
  2. Establish sync chain between DeviceA and DeviceB
  3. Enable Apps sync on DeviceA and DeviceB
  4. DeviceA: Open interia.pl
  5. DeviceA: Click Install Interia in the URL field image
  6. Wait for sync update
  7. DeviceB: Open interia.pl

Actual result:

DeviceB: App is synced
image

DeviceB : App is not installed
image

Expected result:

DeviceB : App is installed
image

Reproduces how often:

100% repro rate

Brave version (brave://version info)

Brave 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS Ubuntu 18.04 LTS

cc @brave/legacy_qa @darkdh @jsecretan @AlexeyBarabash @rebron

@jsecretan
Copy link

@AlexeyBarabash could you also file an upstream bug for this and see what they say?

@AlexeyBarabash
Copy link
Contributor

@jsecretan yes, will do

@AlexeyBarabash
Copy link
Contributor

@btlechowski @jsecretan
I could not reproduce it .
Got the app on device B.
image

@AlexeyBarabash
Copy link
Contributor

On device B just then Right Button click on the icon INT and choose install - then all works.

@rebron
Copy link
Collaborator

rebron commented Sep 18, 2020

@darkdh Is this an upstream bug?
@AlexeyBarabash I'm not sure what you mean here, @btlechowski is saying that it should be installed without having to do a right click on the icon to install. He's going to the url again and seeing a prompt to re-install the app and he shouldn't see that prompt at all.

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Sep 18, 2020
@darkdh
Copy link
Member

darkdh commented Sep 18, 2020

We already had this conversation in slack
https://bravesoftware.slack.com/archives/C2HJYB45N/p1598556672031500
And according to the code in chrome/browser/apps/app_service/app_icon_factory.h , it is intentional

  kPendingLocalLaunch = 0x20,  // Apps that are installed through sync, but
                               // have not been launched locally yet. They
                               // should appear gray until they are launched.

@AlexeyBarabash
Copy link
Contributor

Closing as we have #11911 .

@rebron rebron moved this to Completed in Sync - All platforms Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed
Development

No branches or pull requests

5 participants