Skip to content

feat(tailscale): add status command#9186

Merged
pernielsentikaer merged 8 commits intoraycast:mainfrom
erics118:tailscale/status
Nov 13, 2023
Merged

feat(tailscale): add status command#9186
pernielsentikaer merged 8 commits intoraycast:mainfrom
erics118:tailscale/status

Conversation

@erics118
Copy link
Contributor

@erics118 erics118 commented Nov 9, 2023

Description

adds a command to show the tailscale status
shows the hostname and the connected exit node if there is one

Screencast

CleanShot 2023-11-09 at 15 22 44

Checklist

update deps
update config files
modernize/cleanup code
format
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: tailscale Issues related to the tailscale extension OP is contributor The OP of the PR is a contributor of the extension labels Nov 9, 2023
@raycastbot
Copy link
Collaborator

raycastbot commented Nov 9, 2023

Thank you for your contribution! 🎉

🔔 @samlinville @rosszurowski @samlinville @itsmingjie @erics118 you might want to have a look.

@pernielsentikaer
Copy link
Collaborator

Thanks for your contribution @erics118 🔥

@samlinville do you mind checking this?

@pernielsentikaer pernielsentikaer self-assigned this Nov 10, 2023
Copy link
Contributor

@samlinville samlinville left a comment

Choose a reason for hiding this comment

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

Thanks! Left one request in comments.

no clue how I was testing this command if it never existed in
package.json
Comment on lines +41 to +45
Version: string;
TailscaleIPs: string[];
Self: StatusDevice;
MagicDNSSuffix: string;
Peer: Record<string, StatusDevice>;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Props were reordered to match the order given by the CLI

Copy link
Contributor

@rosszurowski rosszurowski left a comment

Choose a reason for hiding this comment

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

Thanks for the change. There's a few small things that would be nice to clean up before merging, but afterwards this is good to go!

@pernielsentikaer
Copy link
Collaborator

Is this ready for @rosszurowski to have another look @erics118

@rosszurowski
Copy link
Contributor

@pernielsentikaer I'll leave it to @erics118 to confirm, but the code LGTM! All the issues got addressed.

@erics118
Copy link
Contributor Author

All good now!

@pernielsentikaer pernielsentikaer merged commit 22f96bd into raycast:main Nov 13, 2023
@raycastbot
Copy link
Collaborator

Published to the Raycast Store:
https://raycast.com/tailscale/tailscale

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

@erics118 erics118 deleted the tailscale/status branch November 13, 2023 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: tailscale Issues related to the tailscale extension OP is contributor The OP of the PR is a contributor of the extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants