Skip to content

Conversation

@chrisscott
Copy link
Contributor

Copy link
Contributor

@cyx cyx 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 doing this!

@chrisscott chrisscott merged commit d455924 into main Jan 25, 2021
@chrisscott chrisscott deleted the feature/connections-list branch January 25, 2021 21:29
}

func (v *connectionView) AsTableHeader() []string {
return []string{"Name", "Type", "ClientID"}
Copy link

Choose a reason for hiding this comment

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

should this be Connection ID?

@chrisscott
Copy link
Contributor Author

🤦 good catch. Thanks, I'll put in a patch.

@chrisscott
Copy link
Contributor Author

Fixed in #27

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.

4 participants