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

Document RestResponse<T> in rest client #43620

Closed
maxandersen opened this issue Oct 1, 2024 · 5 comments · Fixed by #43759
Closed

Document RestResponse<T> in rest client #43620

maxandersen opened this issue Oct 1, 2024 · 5 comments · Fixed by #43759

Comments

@maxandersen
Copy link
Member

Description

I've multiple times has the need to do more "rest full" behaviors that needed access to response headers ( to get etag ) etc.

turns out that the rest client supports things like:

 @GET
        @Path("/public/schedules")
        RestResponse<ScheduleLinks> getSchedules();

but its not documented at all at https://quarkus.io/guides/rest-client

discussed a bit here https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Getting.20headers.20like.20etag.20via.20rest.20client.20.3F

opening this to have a trace of it and when someone (even me) have time to document it feel encouraged to do so :)

Implementation ideas

No response

Copy link

quarkus-bot bot commented Oct 1, 2024

You added a link to a Zulip discussion, please make sure the description of the issue is comprehensive and doesn't require accessing Zulip

This message is automatically generated by a bot.

Copy link

quarkus-bot bot commented Oct 1, 2024

/cc @cescoffier (rest-client), @geoand (rest-client)

@geoand geoand assigned geoand and unassigned geoand Oct 1, 2024
@geoand
Copy link
Contributor

geoand commented Oct 1, 2024

cc @lasteris

@lasteris
Copy link
Contributor

lasteris commented Oct 1, 2024

I will do it

@geoand
Copy link
Contributor

geoand commented Oct 1, 2024

🙏🏽

@geoand geoand changed the title document existene of RestResponse<T> in rest client Δocument existene of RestResponse<T> in rest client Oct 1, 2024
@geoand geoand changed the title Δocument existene of RestResponse<T> in rest client Document existene of RestResponse<T> in rest client Oct 1, 2024
@cescoffier cescoffier changed the title Document existene of RestResponse<T> in rest client Document RestResponse<T> in rest client Oct 3, 2024
@quarkus-bot quarkus-bot bot added this to the 3.16 - main milestone Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants