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: Docusaurus Build by Upgrading #358

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GatienBoquet
Copy link

What?

  • Upgraded Docusaurus to the latest version.
  • Updated dependencies to ensure compatibility with the new Docusaurus version.
  • Adjusted configuration settings in docusaurus.config.js to align with the updated version requirements.

Why?

Deployments on the web are currently failing, and changes made to the docs are not reflecting on the actual website. Upgrading Docusaurus is intended to resolve these build failures and improve the stability and performance of our documentation site. Additionally, ensuring the Algolia app_id is correctly configured is crucial for the search functionality.

How?

  • Upgraded Docusaurus by modifying the version in the package.json file.
  • Updated all necessary dependencies to match the new Docusaurus requirements.
  • Made required adjustments to the configuration file (docusaurus.config.js) to support the upgraded version.

Regression

  • Fonts: The old fonts are not working with the new version, so they were deleted. Please let me know if you still want to use the old fonts, and we can explore alternative solutions to reintegrate them.

Testing?

  • Verified successful site build with the upgraded Docusaurus version.
  • Checked that all existing documentation pages render correctly.
  • Confirmed that the main functionality of the site is intact post-upgrade.

Request for Additional Information

@nandorojo, I need the actual app_id of Algolia to update the search configuration in docs/docusaurus.config.js. Please provide this information so that we can ensure the search functionality is correctly configured.

Copy link

vercel bot commented Aug 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
moti ❌ Failed (Inspect) Aug 8, 2024 9:59pm

@GatienBoquet
Copy link
Author

Build still fail for some reason.

@nandorojo can you check the vercel logs ?

@nandorojo
Copy link
Owner

i think the issue is the mode version. just pushed a change

@GatienBoquet
Copy link
Author

Great it got deployed on the preview :)

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