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

feat: add opensearch support #980

Merged
merged 4 commits into from Oct 5, 2023
Merged

feat: add opensearch support #980

merged 4 commits into from Oct 5, 2023

Conversation

qwerzl
Copy link
Contributor

@qwerzl qwerzl commented Sep 21, 2023

This PR includes two parts:

  1. Detection for url param q. If detected, open search page automatically.
  2. Add opensearch.xml generation. The link is added to head of each page. The url is configured via opensearch.url in AppConfig.

@Atinux
Copy link
Contributor

Atinux commented Sep 25, 2023

Thank you!

I would be nice to document it, also I think we could move the url to the docus.url part, wdyt?

@qwerzl
Copy link
Contributor Author

qwerzl commented Sep 25, 2023

Thank you!

I would be nice to document it, also I think we could move the url to the docus.url part, wdyt?

I've changed the url.

However I think as docus.url is not a new config key, there's no need for us to update docs?

@Atinux Atinux merged commit 1dfc769 into nuxt-themes:main Oct 5, 2023
1 check passed
Copy link
Contributor

Atinux commented Oct 5, 2023

Thank you!

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.

None yet

2 participants