Skip to content
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

Extended OnlineClient information variables #42

Closed
wants to merge 1 commit into from
Closed

Extended OnlineClient information variables #42

wants to merge 1 commit into from

Conversation

bdeb1337
Copy link
Contributor

Hello,

First off thank you for this great library, which is working great for a project of mine.

I was having somewhat the same request as @30 of wanting to expand the basic clientlist command. I've read the latest serverquery documentation that ships with ts3 server version 3.13.7 and included all parameters regarding this command:

image

I haven't included servergroups yet, since trying to map it as a []int didn't immediately work. I guess it would have something to do with helpers.go line 74 which states // TODO(steve): support groups but I could be wrong. If you are OK with this I could try to give it a go later in a separate pull request? I feel like it might be a little much for one PR otherwise and I could use some guidance in the right direction for that change.

Kind regards

Bob

@bdeb1337
Copy link
Contributor Author

Opening PR on a branch.

@bdeb1337 bdeb1337 closed this Aug 30, 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.

1 participant