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

fixed getList implementation to handle page parsing #20

Merged
merged 7 commits into from
Jan 16, 2025
Merged

Conversation

GW1708
Copy link
Collaborator

@GW1708 GW1708 commented Jan 10, 2025

This PR adresses Issue #19.
The previous implementation attempted to parse a list instead of a page, leading to incorrect behavior.

The previous implementation attempted to parse a list instead of a page, leading to incorrect behavior.
@GW1708 GW1708 self-assigned this Jan 10, 2025
…s right now.

In the future the tests might be changed to abstract classes with the option to choose the server to tests against.
@mjacoby mjacoby marked this pull request as ready for review January 16, 2025 13:01
@mjacoby mjacoby merged commit 84986e2 into main Jan 16, 2025
3 checks passed
@mjacoby mjacoby deleted the bugfix-getAll branch January 16, 2025 13:54
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