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

Remove "agrest-client" #527

Closed
andrus opened this issue Jan 9, 2022 · 0 comments
Closed

Remove "agrest-client" #527

andrus opened this issue Jan 9, 2022 · 0 comments
Milestone

Comments

@andrus
Copy link
Contributor

andrus commented Jan 9, 2022

As a part of stack refactoring, we are going to simplify our modules architecture by removing "agrest-client". "agrest-client" hasn't been supported for years and is falling behind. And Agrest can be queried directly via HTTP, and a special client doesn't provide much value.

The most valuable (and less trivial) piece of the client is the ability to read the data response as ClientDataResponse<T>, with T objects being actually created. We might eventually provide a utility method in Agrest to achieve that without the client runtime.

Would have been great if server-side DataResponse could have been easy to deserialize from the client, but since it contains encoders and does not contain the total, it is not as easy to use as a POJO.

andrus added a commit that referenced this issue Jan 9, 2022
@andrus andrus added this to the 5.0 milestone Jan 9, 2022
@andrus andrus closed this as completed Jan 9, 2022
andrus added a commit that referenced this issue Jan 9, 2022
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

No branches or pull requests

1 participant