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

Update Sync All Facilities Modal to allow unregistered facilities to sync #10758

Closed
LianaHarris360 opened this issue May 30, 2023 · 0 comments · Fixed by #10765
Closed

Update Sync All Facilities Modal to allow unregistered facilities to sync #10758

LianaHarris360 opened this issue May 30, 2023 · 0 comments · Fixed by #10765
Assignees
Labels
APP: Device Re: Device App (content import/export, facility-syncing, user permissions, etc.) P1 - important Priority: High impact on UX

Comments

@LianaHarris360
Copy link
Member

Observed behavior

When pressing Sync All to open Sync all facility data modal, the sync submission is disabled with the tooltip message 'There are no registered facilities on this device'. Because facilities could already be registered and may not know until they are synced, the sync button should not be disabled and the tooltip message should be removed..

The message 'You must be connected to the internet' is also incorrectly displayed.

Screenshot 2023-05-30 at 11 58 07 AM

Expected behavior

When Sync All button is pressed, the sync button should not be disabled on the opened modal. If user is connected to the internet, the message 'You must be connected to the internet' should not be displayed.

User-facing consequences

Users are unable to sync unregistered facilities when clicking Sync all.

Steps to reproduce

  1. Navigate to Device > Facilities
  2. Select Sync all
  3. If facility is unregistered, the sync button is disabled and on hover, the message 'There are no registered facilities on this device' is displayed
@LianaHarris360 LianaHarris360 added P1 - important Priority: High impact on UX APP: Device Re: Device App (content import/export, facility-syncing, user permissions, etc.) labels May 30, 2023
@akolson akolson self-assigned this Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Device Re: Device App (content import/export, facility-syncing, user permissions, etc.) P1 - important Priority: High impact on UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants