Skip to content

Direct users to the Network Status dialog#1831

Merged
jdeanwallace merged 2 commits into
masterfrom
direct-users-to-network-status-dialog
Aug 27, 2024
Merged

Direct users to the Network Status dialog#1831
jdeanwallace merged 2 commits into
masterfrom
direct-users-to-network-status-dialog

Conversation

@jdeanwallace
Copy link
Copy Markdown
Contributor

@jdeanwallace jdeanwallace commented Aug 1, 2024

Resolves #1826

This PR adds a convenience button that opens the Network Status dialog after a user configures their Wi-Fi credentials.

Additionally, we've added a hint to the tooltip of the <connection-indicator> to direct users to the Network Status dialog.

Demo video:

Screen.Recording.2024-08-16.at.13.57.53.mov

Review on CodeApprove

@jdeanwallace jdeanwallace requested a review from db39 August 1, 2024 20:28
Copy link
Copy Markdown
Contributor Author

Automated comment from CodeApprove ➜

@db39 please review this Pull Request

@jdeanwallace
Copy link
Copy Markdown
Contributor Author

@db39 - Just checking if this is still on your radar?

1 similar comment
@jdeanwallace
Copy link
Copy Markdown
Contributor Author

@db39 - Just checking if this is still on your radar?

@db39
Copy link
Copy Markdown
Contributor

db39 commented Aug 27, 2024

@jdeanwallace - Sorry, I completely missed this, I'm not sure why I didn't get the notifications before. Thanks for the ping.

Copy link
Copy Markdown
Contributor

@db39 db39 left a comment

Choose a reason for hiding this comment

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

Automated comment from CodeApprove ➜

Approved on CodeApprove
✔️ Approved

LGTM!


👀 @jdeanwallace it's your turn please take a look

@jdeanwallace jdeanwallace merged commit b3be4fa into master Aug 27, 2024
@jdeanwallace jdeanwallace deleted the direct-users-to-network-status-dialog branch August 27, 2024 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Direct users to Network Status dialog

2 participants