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

Fix "Sorry, no guides matched your search." when starting initial search and missing CSS classes when guide search defaults to Javascript #1911

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere requested a review from gsmet March 11, 2024 16:33
@yrodiere yrodiere force-pushed the fix-initial-no-results branch from 4ac8b8e to 83658aa Compare March 11, 2024 16:34
Copy link

github-actions bot commented Mar 11, 2024

😭 Deploy PR Preview failed.

@gsmet
Copy link
Member

gsmet commented Mar 12, 2024

I'm not entirely sure of what's going on but when testing locally, I have Loading... appearing and then nothing on the network and no search results appearing, just Loading... forever.

@yrodiere
Copy link
Member Author

I'm not entirely sure of what's going on but when testing locally, I have Loading... appearing and then nothing on the network and no search results appearing, just Loading... forever.

If you test locally, you also need to launch search.quarkus.io locally.

That being said... javascript search should have stepped in :/

I'll have a look.

@yrodiere yrodiere force-pushed the fix-initial-no-results branch from 83658aa to a3c5436 Compare March 12, 2024 09:20
@yrodiere yrodiere changed the title Fix "Sorry, no guides matched your search." when starting initial search Fix "Sorry, no guides matched your search." when starting initial search and missing CSS classes when guide search defaults to Javascript Mar 12, 2024
@yrodiere
Copy link
Member Author

@gsmet should be all good now. I also fixed the problem of missing CSS classes when defaulting to Javascript search.

@yrodiere yrodiere merged commit e4a2ffb into quarkusio:develop Mar 15, 2024
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.

When typing something in search after first load, message saying sorry no guides appears
2 participants