Skip to content

fix(search): bump @docsearch/js to 3.3 & add appId in config#159

Merged
vfdev-5 merged 17 commits intomainfrom
fix/algolia-search
Dec 19, 2022
Merged

fix(search): bump @docsearch/js to 3.3 & add appId in config#159
vfdev-5 merged 17 commits intomainfrom
fix/algolia-search

Conversation

@jeffydc
Copy link
Collaborator

@jeffydc jeffydc commented Nov 8, 2022

Algolia docsearch has moved to new version and let admins directly edit the config via their web interface. So we need to update the docsearch version we are using to the latest stable one as well.

@netlify
Copy link

netlify bot commented Nov 8, 2022

Deploy Preview for pytorch-ignite ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-ignite/deploys/637096ab772b900340b33290
😎 Deploy Preview https://deploy-preview-159--pytorch-ignite.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jeffydc jeffydc changed the title fix(search): bump @docsearch/js to 3.3 & add appId in config [Draft] fix(search): bump @docsearch/js to 3.3 & add appId in config Nov 8, 2022
@vfdev-5
Copy link
Member

vfdev-5 commented Nov 8, 2022

@ydcjeff thanks for the PR !
Concerning failing linkcheck, I fixed the one with news_commentary. Do you think we can do anything with https://towardsdatascience.com ? Can we setup the checker to ignore them ?

@jeffydc
Copy link
Collaborator Author

jeffydc commented Nov 11, 2022

check-ipynb-to-md is expected to fail since it is blocked by pytorch-ignite/examples#83

@jeffydc jeffydc changed the title [Draft] fix(search): bump @docsearch/js to 3.3 & add appId in config fix(search): bump @docsearch/js to 3.3 & add appId in config Nov 13, 2022
@jeffydc
Copy link
Collaborator Author

jeffydc commented Nov 23, 2022

@vfdev-5 the submodule failure is fixed in this PR and netlify failure is not updated in the check although the site is built successfully. https://deploy-preview-159--pytorch-ignite.netlify.app/

@vfdev-5
Copy link
Member

vfdev-5 commented Dec 19, 2022

@ydcjeff can we merge this PR ?

@jeffydc
Copy link
Collaborator Author

jeffydc commented Dec 19, 2022

@vfdev-5 yes, it's ready.

@vfdev-5 vfdev-5 merged commit 01b2522 into main Dec 19, 2022
@vfdev-5 vfdev-5 deleted the fix/algolia-search branch December 19, 2022 12:23
jeffydc pushed a commit that referenced this pull request Dec 30, 2022
Follow up of #159 , I forgot to update the css version.
vfdev-5 pushed a commit that referenced this pull request Jan 5, 2023
Follow up of #159 , I forgot to update the css version.
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