Cht-core: Reloading the app while offline shows sync status "required"

Created on 14 Mar 2019  路  6Comments  路  Source: medic/cht-core

Steps to reproduce:

  1. Load the app
  2. Fully sync so the hotdog menu says "all reports synced"
  3. Go offline
  4. Refresh the app
  5. Open the hotdog menu

What should happen:

  • No translation warnings in the console
  • The sync status is "unknown"

What actually happens:

  • A console warning is shown
  • The sync status is "reports to sync"

Console log:

Translation for sync.status.undefined doesn't exist

Screenshot from 2019-03-14 16-46-23

Environment:

  • Instance: localhost
  • Browser: Chrome
  • Client platform: Ubuntu
  • App: webapp
  • Version: 3.5.0 (head of master)
3 - Low Bug

All 6 comments

AT: Testing this depends on #5481 or #4898 being fixed, because currently there's basically always a feedback doc to submit, so instead of seeing "unknown" you'll see "error".

Ready for AT in 5501-rationalise-sync-events

Can you look at the conflicts on this branch @garethbowen ?https://github.com/medic/medic/pull/5503/conflicts

@ngaruko I've fixed the conflicts - once the build passes you should be good to go again.

I'm seeing this work on the web currently but waiting on the fixes for working in the android-app so we can verify on phone .

LGTM -
image

Was this page helpful?
0 / 5 - 0 ratings