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

[Breaking][Requires Quarkus 3.12+] Use new generated static resources #686

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

ia3andy
Copy link
Collaborator

@ia3andy ia3andy commented Jun 13, 2024

With this change Quinoa is finally able to delegate file server to Quarkus Vert.x http using the new GeneratedStaticResourceBuildItem.

  • Requires Quarkus 3.12+
  • [Breaking] quarkus.quinoa.index-page has been removed. Use quarkus.http.static-resources.index-page instead.

@ia3andy
Copy link
Collaborator Author

ia3andy commented Jun 13, 2024

cc @mcruzdev your new stuff again :)

@ia3andy ia3andy requested a review from melloware June 13, 2024 09:09
@ia3andy
Copy link
Collaborator Author

ia3andy commented Jun 13, 2024

@melloware since this is for 3.12+ maybe we should start a new branching for backports?

@melloware melloware added this to the 2.4.0 milestone Jun 13, 2024
@melloware
Copy link
Contributor

I will cut the 2.3.8 release and make a 2.3.X branch so we can start this new version as 2.4.0

@melloware melloware force-pushed the generated-static-resources branch from bd1dddc to b2aed60 Compare June 13, 2024 11:34
Copy link
Contributor

@melloware melloware left a comment

Choose a reason for hiding this comment

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

LGTM. I just rebased this branch and I made a new 2.3.X branch for 3.8 LTS and I will make main now 2.4.X on for Quarkus 3.12 and higher

@melloware melloware added the enhancement New feature or request label Jun 13, 2024
@melloware melloware force-pushed the generated-static-resources branch from b2aed60 to 1e59a8b Compare June 13, 2024 11:59
@melloware melloware merged commit 5ca8d25 into quarkiverse:main Jun 13, 2024
2 checks passed
import static io.quarkiverse.quinoa.deployment.config.QuinoaConfig.isDevServerMode;
import static io.quarkiverse.quinoa.deployment.config.QuinoaConfig.isEnabled;
import static io.quarkiverse.quinoa.deployment.config.QuinoaConfig.toHandlerConfig;
import static io.quarkiverse.quinoa.deployment.config.QuinoaConfig.*;
Copy link
Member

Choose a reason for hiding this comment

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

Miss .* import

Copy link
Contributor

Choose a reason for hiding this comment

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

i am fixing in another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants