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 pointer to show initial sync is in progress on the map view #3544

Open
2 tasks
AnnieMungai opened this issue Oct 7, 2024 · 2 comments
Open
2 tasks
Assignees
Labels
EUSM Sync enhancement Enhancements related to sync

Comments

@AnnieMungai
Copy link
Contributor

AnnieMungai commented Oct 7, 2024

Describe the feature request.
When conducting a sync on EUSM app, after selecting locations, on the map view the sync bar shows for a few seconds but then disappears. The usr is thus not able to know whether the sync is still in progress. One can simply think that the app is frozen and not doing anything.

The ask: Have a UI that let's the user know that the initial sync still progressing and they should give it some time.

Screenrecorder-2024-10-07-17-30-30-591.mp4

Additional context

  1. The map view is the default view on the EUSM app.
  2. On the register view, there is a progress bar when the sync is in progress.

Acceptance criteria

  • There is a UI on the map view that lets the user know that sync is in progress. This should be intuitive.
  • User on the map view is notified when all the service points have been downloaded. i.e. sync is completed.

Area path
A list of ordered steps in the app on usage of the feature to support anyone testing it e.g. Code reviewer, QA e.g.

  1. Login to the app with a username and password.
  2. Select the locations as directed by the app.
  3. Tap on Sync at the bottom of the selection panel.
  4. Observe.

Implementation plan (For Engineers)
The plan for implementing the solution e.g. via a description or a check list for the various ordered tasks that will need to be completed.
i.e. Describe how you intend to solve the problem

@AnnieMungai AnnieMungai added EUSM Sync enhancement Enhancements related to sync labels Oct 7, 2024
@dubdabasoduba
Copy link
Member

@AnnieMungai we had a chat with part of the Product team today. We have a go-ahead to add the spinner. @HenryRae @ellykits should we use the same spinner we have on the registers? i.e. the white background with a spinner in the centre

@ellykits
Copy link
Collaborator

This happens because the custom resource (LocationHierarchy) sync does not use the SDK's Sync worker that transmits SyncJobStatuses. See the discussion I started in the SDK repo @dubdabasoduba google/android-fhir#2625.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EUSM Sync enhancement Enhancements related to sync
Projects
None yet
Development

No branches or pull requests

3 participants