-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Fixes incorrect rel=self web link #38265
Conversation
bpasson
commented
Jan 17, 2024
•
edited by geoand
Loading
edited by geoand
- Fixes Incorrect web links Quarkus Hibernate Reactive Rest Data Panache #38247
This comment was marked as resolved.
This comment was marked as resolved.
This comment has been minimized.
This comment has been minimized.
🙈 The PR is closed and the preview is expired. |
This comment has been minimized.
This comment has been minimized.
Done! |
...s/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksProcessor.java
Outdated
Show resolved
Hide resolved
...s/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksProcessor.java
Show resolved
Hide resolved
...s/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksProcessor.java
Outdated
Show resolved
Hide resolved
...st/java/io/quarkus/resteasy/reactive/links/deployment/RestLinksWithFailureInjectionTest.java
Outdated
Show resolved
Hide resolved
...est/java/io/quarkus/resteasy/reactive/links/deployment/TestRecordWithIdAndPersistenceId.java
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
@gsmet I'm currently having too much on my table and would need quite some time getting into the topic. Sorry. |
Co-authored-by: Stéphane Épardaud <[email protected]>
Co-authored-by: Stéphane Épardaud <[email protected]>
✔️ The latest workflow run for the pull request has completed successfully. It should be safe to merge provided you have a look at the other checks in the summary. You can consult the Develocity build scans. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@FroMage next time make sure the commits are properly squashed before merging. Thanks! |