You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by GitBox <gi...@apache.org> on 2018/08/21 00:06:59 UTC

[GitHub] shazron commented on issue #874: Documentation search is not working

shazron commented on issue #874: Documentation search is not working
URL: https://github.com/apache/cordova-docs/issues/874#issuecomment-414504040
 
 
   The doc search has been fixed by Algolia (thanks @s-pace!), but we need to make some changes.
   From @s-pace:
   
   - I have updated [your configuration in order to avoid this situation to happen again](https://github.com/algolia/docsearch-configs/commit/1f598ac78525004d0a114ffd7e50ffec364b655c). I have also deployed these changes which are live, you can check it from your website, the search is back.
   
   In order to prevent such behaviour in the future, we would need you to implement some changes from your pages. We do not want to only rely on the URLs itself. Every page's content should have enough context.
   - [ ] We would need you to add some meta tags within your html. These tags should be built this way:
   ```html
   <meta name="docsearch:language" content="en">
   <meta name="docsearch:version" content="8.x">
   ```
   Please note that the `content` attribute should be equal to the one used from the DocSearch-UI JS snippet. For example on this page:
   ```javascript
   <script type="text/javascript">
   docsearch({
   apiKey: '0a916ab198bd93d031aa70611271e42e',
   indexName: 'cordova',
   inputSelector: '#header-search-field',
   algoliaOptions: { 'facetFilters': ["version: 8.x", "language: en"] }
   });
   </script>
   ```
   This is something you can find on the Docusaurus website or Jest (implementing docusaurus) as a detailed example.
   
   - [ ] We would need your sitemap to be updated. It will be our main source of truth in order to discover your website. We didn't detected any issue regarding this sitemap so far.
   - [ ] Avoid having pages duplicates pages (e.g. this page and this index.html one). You might want to add a 3XX status to one of both. We avoid such issue by using the stop_urls parameter. This point is solved by preventing crawling the URLs matching the following regex: "https://cordova.apache.org/docs/[^/]+/[^/]+/index.html$"
   - [ ] PR your configuration as soon as you are ready with these changes. We will merge it and deploy it once it is live. The current configuration will work with the above changes anyway. You can find the expected configuration enclosed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org