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

Adjust the condition of not returning index.html #398

Merged

Conversation

marko-bekhta
Copy link
Collaborator

to not rely on launch mode but on profiles instead. This way staging, that also runs in a normal launch mode can benefit from the index search page

Copy link
Member

@yrodiere yrodiere left a comment

Choose a reason for hiding this comment

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

Ha. Talk about a hidden feature. I wonder why this specific observer is in this (rather unrelated) class.

Ideally we'd rather have a dedicated class, and would set this through config (e.g. search.ui.enabled), but... this will do. Thank you!

@marko-bekhta
Copy link
Collaborator Author

search.ui.enabled

that should be doable ... let me update the PR 😃

@marko-bekhta marko-bekhta force-pushed the fix/make-ui-available-for-staging branch from ad6d459 to 9ce0b6a Compare January 28, 2025 15:47
To rely on an application property that disables the UI in the prod profile
@marko-bekhta marko-bekhta force-pushed the fix/make-ui-available-for-staging branch from 9ce0b6a to 505fc5f Compare January 28, 2025 15:59
@marko-bekhta marko-bekhta merged commit f63fd1d into quarkusio:main Jan 28, 2025
1 check passed
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