Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

Fix/14578 no internet prevents clicking on srs #5019

Merged

Conversation

puneetmahali1
Copy link
Contributor

Description

No internet connections prevents clicking on Self test submission

Link to Jira

https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-14578

Screenshots

EXPOSUREAPP-14578.MP4

@puneetmahali1 puneetmahali1 added the bug Something isn't working label Jan 24, 2023
@puneetmahali1 puneetmahali1 added this to the v3.1.0 milestone Jan 24, 2023
@puneetmahali1 puneetmahali1 requested a review from a team January 24, 2023 14:54
@Ein-Tim
Copy link
Contributor

Ein-Tim commented Jan 24, 2023

So IIUC when you are not connected to the internet and click on the tile nothing happens. Like nothing at all? I don't think that's a good idea! Some sort of error should be shown to the user, otherwise this could lead to very confused users.

@thomasaugsten
Copy link
Member

@Ein-Tim This is under certain circumstances the situation with 3.0 and this PR will fix the issue.

@Ein-Tim
Copy link
Contributor

Ein-Tim commented Jan 24, 2023

@thomasaugsten So there will actually be some sort of UI information presented to the user why this doesn't work?

@thomasaugsten
Copy link
Member

Depends if he never has an internet connection he will receive the error that we cannot check his phone time.
If its a temporary no internet he will receive an no internet error when he tries to submit

@30mar 30mar self-requested a review January 25, 2023 13:20
Copy link
Contributor

@30mar 30mar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aren't we doing a shared class for checking the network now as discussed?

@puneetmahali1 puneetmahali1 merged commit f19affa into release/3.1.x Jan 26, 2023
@puneetmahali1 puneetmahali1 deleted the fix/14578-not-internet-prevents-srs-clicking branch January 26, 2023 09:30
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants