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

No UI prompt of update available or to relaunch to install update #877

Closed
srirambv opened this issue Aug 29, 2018 · 4 comments
Closed

No UI prompt of update available or to relaunch to install update #877

srirambv opened this issue Aug 29, 2018 · 4 comments
Labels
closed/works-for-me design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/updater

Comments

@srirambv
Copy link
Contributor

Description

No UI prompt of update available or to relaunch to install update

Steps to Reproduce

  1. Install an older version
  2. Wait till a new version is released
  3. No UI indication of update available unless you go to chrome://settings/help it doesn't prompt to update and relaunch

Actual result:

No UI prompt of update available or to relaunch to install update

Expected result:

Should show some indication that update is available and a restart is pending to install it

Reproduces how often:

100%

Brave version (about:brave info)

All version

Reproducible on current release:

No. Any update shows a notification bar to install update or defer until restart

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Chrome?

Additional Information

cc: @rossmoody @bradleyrichter @bbondy

@srirambv srirambv added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/updater labels Aug 29, 2018
@srirambv srirambv added this to the Releasable builds 0.55.x milestone Aug 29, 2018
@simonhong
Copy link
Member

We are using chrome's update strategy for now.
Brave will install newer version in background(with some hours interval checking) and wait users re-launch.
When newer version install is finished, brave does not immediately inform to users. (It will distract user)
Instead, it waits some times then informs to users by using app menu.

@bbondy Do we have a plan for adding our own update inform UI?

@bbondy
Copy link
Member

bbondy commented Aug 30, 2018

I think we might have something already if it is marked as an urgent release? In any case as long as it updates in the background and when going to about, that is all we need for initial release.

@bbondy bbondy modified the milestones: Releasable builds 0.55.x, Backlog Aug 30, 2018
@simonhong
Copy link
Member

simonhong commented Aug 30, 2018

I tested is critical check box option and this feature does nothing.

UpgradeDetectorImpl determines whether currently install version in background is critical or not by checking cpv register value. (GetCurrentlyInstalledVersionImpl(&critical_update);)
When new version (is critical checked) is installed, our updater (omaha) doesn't set cpv to registry.
Also, I can't find any cpv related code in omaha project.

CC: @RyanJarv

@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@bsclifton
Copy link
Member

Closing as I don't think this is a problem anymore. Update icon is shown on Hamburger menu when update is available 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/works-for-me design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/updater
Projects
None yet
Development

No branches or pull requests

5 participants