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

Adds a number of helper structs for z responses #6249

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

ripienaar
Copy link
Contributor

These are used by the cli and other tools to decode the responses from the z endpoints

Signed-off-by: R.I.Pienaar [email protected]

@ripienaar ripienaar requested a review from a team as a code owner December 13, 2024 08:01
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

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

Are these different from what we really use?
Could we create an alias instead?

@ripienaar
Copy link
Contributor Author

These types doesn't exist anywhere its constructed from a any wrapped into all in the same structure

So to use them everyone either have to double json decode (This is how MH designed it and insisted its used) or write these structs themselves, so this just helps out.

@derekcollison
Copy link
Member

ok let's comment why they are here and not used in any of the server code base just for clairity.

These are used by the cli and other tools to decode
the responses from the z endpoints

Signed-off-by: R.I.Pienaar <[email protected]>
@ripienaar
Copy link
Contributor Author

ok let's comment why they are here and not used in any of the server code base just for clairity.

Have clarified the comment that was there already a bit further

@derekcollison derekcollison self-requested a review December 14, 2024 16:36
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

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

LGTM

@derekcollison derekcollison merged commit 5431acb into nats-io:main Dec 14, 2024
2 checks passed
neilalexander added a commit that referenced this pull request Dec 16, 2024
Includes the following:

- #6260
- #6249
- #6264
- #6265
- #6266

Signed-off-by: Neil Twigg <[email protected]>
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.

2 participants