-
Notifications
You must be signed in to change notification settings - Fork 1.3k
No network check for saving tags #16737
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 network check for saving tags #16737
Conversation
Generated by 🚫 dangerJS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carl-duncan Thanks for working on this Issue.
I can see the same issue when I follow the below steps
- Go to My Site -> Site Settings
- Enable Airplane Mode
- Go to Tags
- Click at create Tag and Enter tag details
- Notice that on pressing add tag the screen freezes
vidma_recorder_15062022_124545.mp4
Let me know if you need any additional info
Thank you I will check it out! |
@AjeshRPai Hey , fixed the issue for that specific interaction. |
@carl-duncan Thanks for making the change. I feel now the behaviour is not user-friendly. When the user clicks on a setting, I would say it's better to show a snack bar with no connection message rather than closing the site settings page itself. The user might be unable to figure out what went wrong there as there are items like Site Title, Tag line etc., which we can edit without the internet. WDYT? |
Thank you again for the review, Yeah I agree, It would be a better user experience and easier for them to take their next step to fix the connection and continue. |
d179a77
to
0b754c2
Compare
Hey @AjeshRPai , I added a toast to display when a user is trying to save a toast when there is no connection. I think that would be the best course of action as users will still have easy access to create a tag once they get connectivity and not break the UI. |
@carl-duncan I have tested the latest changes. I still see the freezing issue when I try to add a tag without internet. I am not able to see any toast with an internet connection error. Please find the below video capture showing the behaviour. Screen_Recording_20220620-192050_WordPress.Pre-Alpha.mp4I believe the better fix would be to show a no internet toast on clicking at the Tags/Timezone settings if we don't have internet. No need to finish the activity but show the No Internet connection toast. WDYT?. 🤔 |
Hey @AjeshRPai could you retest with mobile data off as well? |
Hey @carl-duncan , Sorry for the late reply.
Same behaviour as above in the Video capture. The screen freezes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the issue still persists. Please look at the above comments for clarification.
Fixes #15590
Added the network check to onResume
To test:
Go into site settings
Open the tags
Go into airplane mode
Go into the app and close the tags section
Regression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR submission checklist:
RELEASE-NOTES.txt
if necessary.