Skip to content

Conversation

carl-duncan
Copy link
Contributor

@carl-duncan carl-duncan commented Jun 13, 2022

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

  1. Potential unintended areas of impact

  2. What I did to test those areas of impact (or what existing automated tests I relied on)

  3. What automated tests I added (or what prevented me from doing so)

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 13, 2022

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

Copy link
Contributor

@AjeshRPai AjeshRPai left a 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

@carl-duncan
Copy link
Contributor Author

Thank you I will check it out!

@carl-duncan
Copy link
Contributor Author

carl-duncan commented Jun 15, 2022

@AjeshRPai Hey , fixed the issue for that specific interaction.

@carl-duncan carl-duncan requested a review from AjeshRPai June 16, 2022 13:29
@AjeshRPai
Copy link
Contributor

@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?

@carl-duncan
Copy link
Contributor Author

carl-duncan commented Jun 17, 2022

@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.

@carl-duncan carl-duncan force-pushed the fix/15590-no-network-check-in-site-settings branch from d179a77 to 0b754c2 Compare June 17, 2022 13:55
@carl-duncan carl-duncan reopened this Jun 17, 2022
@carl-duncan carl-duncan changed the title No Network check in site settings No network check for saving tags Jun 17, 2022
@carl-duncan
Copy link
Contributor Author

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.

@AjeshRPai
Copy link
Contributor

AjeshRPai commented Jun 20, 2022

@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.mp4

I 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?. 🤔

@carl-duncan
Copy link
Contributor Author

@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.mp4
I 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?

@AjeshRPai
Copy link
Contributor

Hey @carl-duncan , Sorry for the late reply.

Hey @AjeshRPai could you retest with mobile data off as well

Same behaviour as above in the Video capture. The screen freezes.

Copy link
Contributor

@AjeshRPai AjeshRPai left a 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.

@justtwago justtwago closed this Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Site Settings Screen - No Network check
3 participants