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 internet connection modal is not shown when setting up sync after being offline #2468

Closed
btlechowski opened this issue Dec 11, 2018 · 4 comments

Comments

@btlechowski
Copy link

btlechowski commented Dec 11, 2018

Steps to Reproduce

  1. Clean install
  2. Go offline (unplug internet connection from router or disconnect wifi)
  3. Open chrome://bravesync/ and chose I am new to Sync
  4. Click Set up Sync
  5. Go online and wait X minutes (waited for over an hour; TODO determine X)

Actual result:

sync

Device list is empty
No retry is made
Reload does not help
Note: Closing and opening Brave will fix the issue

Expected result:

No internet connection modal

Reproduces how often:

always

Brave version (brave://version info)

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows 7 x64

Also reproducible on Windows 10 x64, when internet connection is unplugged from router
When I turn off wifi on Window 10, I get No internet connection modal.

cc @darkdh @kjozwiak @LaurenWags @srirambv @GeetaSarvadnya

@LaurenWags
Copy link
Member

LaurenWags commented Dec 11, 2018

Did not reproduce on macOS for me, I got an (expected) error message at step 4 when trying to 'Set up Sync' after wifi has been turned off.

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X

screen shot 2018-12-11 at 5 41 36 pm

@btlechowski btlechowski changed the title No retry on setting up sync after being offline No internet connection modal is not shown when setting up sync after being offline Dec 11, 2018
@srirambv
Copy link
Contributor

srirambv commented Dec 12, 2018

Cant reproduce on Linux. Here's what i tried

  • Disconnect network
  • Try creating sync chain, shows appropriate error message No internet connection
  • Bring device back online
  • Dismiss error message and click Setup Sync
  • Creates sync chain and device name shows up within 5 seconds

@bbondy bbondy added the priority/P4 Planned work. We expect to get to it "soon". label Dec 12, 2018
@cezaraugusto
Copy link
Contributor

@btlechowski could you please check again and see if issue persists?

@cezaraugusto cezaraugusto added the needs-investigation A bug not 100% confirmed/fixed label Dec 17, 2018
@kjozwiak kjozwiak added this to the 1.x Backlog milestone Dec 18, 2018
@btlechowski
Copy link
Author

Does not reproduce in:

Brave 0.59.20 Chromium: 72.0.3626.28 (Official Build) beta (64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Windows 7

@cezaraugusto cezaraugusto modified the milestones: 1.x Backlog, Dupe / Invalid / Not actionable Jan 14, 2019
@kjozwiak kjozwiak added closed/invalid and removed OS/Windows QA/Test-Plan-Specified QA/Yes feature/sync needs-investigation A bug not 100% confirmed/fixed priority/P4 Planned work. We expect to get to it "soon". labels Feb 6, 2019
@rebron rebron removed this from the Dupe / Invalid / Not actionable milestone May 10, 2019
@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Jun 3, 2019
@bbondy bbondy removed this from the Dupe / Invalid / Not actionable milestone May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants