Skip to content

fix users endpoint#278

Merged
Enovotny merged 5 commits intomainfrom
update/users
Mar 27, 2026
Merged

fix users endpoint#278
Enovotny merged 5 commits intomainfrom
update/users

Conversation

@Enovotny
Copy link
Copy Markdown
Collaborator

fix users endpoint. add paging, office and user-like selections.

Copy link
Copy Markdown
Collaborator

@krowvin krowvin left a comment

Choose a reason for hiding this comment

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

Not sure if you were waiting on a review

pyproject.toml Outdated
repository = "https://github.com/HydrologicEngineeringCenter/cwms-python"

version = "1.0.3"
version = "1.0.4"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Might have to bump this version again before merging

}
try:
response = api.get("users", params=params, api_version=1)
response = api.get_with_paging(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I forgot this method existed, nice

@sonarqubecloud
Copy link
Copy Markdown

@Enovotny Enovotny merged commit 4cc357c into main Mar 27, 2026
9 checks passed
@Enovotny Enovotny deleted the update/users branch March 27, 2026 21: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