Skip to content

feat: show if using coder connect in status indicator #491

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

Merged
merged 2 commits into from
May 2, 2025

Conversation

ethanndickson
Copy link
Member

@ethanndickson ethanndickson commented May 1, 2025

Relates to #447

Following on from coder/coder#17572, this PR has the extension read the using_coder_connect bool from the network stats file, and change the status accordingly.

image

Copy link
Member Author

ethanndickson commented May 1, 2025

@ethanndickson ethanndickson marked this pull request as ready for review May 1, 2025 04:15
@ethanndickson ethanndickson changed the title feat: add coder connect to connection status indicator feat: show if using coder connect in status indicator May 1, 2025
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

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

Awesome.

Copy link
Member Author

ethanndickson commented May 2, 2025

Merge activity

  • May 1, 9:06 PM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 1, 9:07 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 1, 9:07 PM EDT: @ethanndickson merged this pull request with Graphite.

@ethanndickson ethanndickson force-pushed the ethan/coder-connect-status branch from eb2cd8a to 6bb0b30 Compare May 2, 2025 01:06
@ethanndickson ethanndickson merged commit 1395a5c into main May 2, 2025
3 checks passed
@ethanndickson ethanndickson deleted the ethan/coder-connect-status branch May 2, 2025 01:07
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.

3 participants