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

feat(oss): list users via the API now supports offset, limit or after parameter #44

Merged
merged 1 commit into from
May 12, 2021

Conversation

bednar
Copy link
Contributor

@bednar bednar commented May 7, 2021

Related to influxdata/influxdb#21367
Related to influxdata/influxdb-client-csharp#190

List users via the API now supports offset, limit or after parameter.

Copy link
Contributor

@glinton glinton left a comment

Choose a reason for hiding this comment

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

looks good, pending influxdata/influxdb#21367 merge

@glinton
Copy link
Contributor

glinton commented May 10, 2021

@bednar, do you mind pushing again to trigger the ci tests, thanks

@bednar bednar force-pushed the feat/users-limit-offset-after branch from d9cbdec to 258769d Compare May 11, 2021 07:17
@bednar
Copy link
Contributor Author

bednar commented May 11, 2021

@glinton sources are rebased with master, CI passed, but we are still waiting for influxdata/influxdb#21367

@bednar
Copy link
Contributor Author

bednar commented May 12, 2021

@glinton PR is ready to merge, influxdata/influxdb#21367 is merged into master

@glinton glinton merged commit bf5cc54 into influxdata:master May 12, 2021
@bednar bednar deleted the feat/users-limit-offset-after branch May 13, 2021 07:00
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