Skip to content

Conversation

justinegeffen
Copy link
Contributor

@justinegeffen justinegeffen commented Oct 1, 2025

Added Typesense to the documentation as a search provider.

This has been approved to merge but needs a final security and devops review.

Copy link

netlify bot commented Oct 1, 2025

Deploy Preview for seqera-docs failed. Why did it fail? →

Name Link
🔨 Latest commit 796ec29
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/68e7d6f48c78f600083645fc

@mavi-sqr
Copy link
Contributor

mavi-sqr commented Oct 2, 2025

Hi @justinegeffen, writing here what I shared on Slack, so everything is in the PR just in case we want to double-check someday.

Just show up, I didn't work in the UI.
What I found is that since the desktop header is custom, we need to explicitly import and add the component there. On mobile it shows up by default because the header isn’t custom. (Not a big fan of Docusaurus 🫠). The fix was essentially importing it into the correct file.
In mobile, it was not properly displayed due to a style overwrite.

@justinegeffen justinegeffen dismissed arnaualcazar’s stale review October 8, 2025 18:20

Change was implemented.

@justinegeffen justinegeffen merged commit b3474bf into master Oct 10, 2025
7 checks passed
@justinegeffen justinegeffen deleted the justine-typesense branch October 10, 2025 15:44
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.

3 participants