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

Hotfix: Replaces internal linking with direct links for content through react components #1343

Merged
merged 1 commit into from
Oct 19, 2024

Conversation

fgalz
Copy link
Collaborator

@fgalz fgalz commented Oct 19, 2024

This hotfix resolves the issue with the internal links from react components with the use of the "@docusaurus/Translate"function in the documentation. All affected internal links have been replaced with full direct links to avoid errors in the URL structure caused by the trailing slash problem in productive use. This should eliminate all 404 errors after page reloads for the users and indexing/console with Google.

Disclaimer: This is not intended to be an optimal or final solution. It is a temporary solution until a long-term and better solution is developed.

@fgalz fgalz changed the title Hotfix: Replace internal linking with direct links for Hotfix: Replace internal linking with direct links for content through react components Oct 19, 2024
@fgalz fgalz changed the title Hotfix: Replace internal linking with direct links for content through react components Hotfix: Replaces internal linking with direct links for content through react components Oct 19, 2024
Copy link

badge

Build Successful! You can find a link to the downloadable artifact below.

Name Link
Commit c722e5f
Logs https://github.com/zaphosting/docs/actions/runs/11417945891
Download https://github.com/zaphosting/docs/suites/29831262301/artifacts/

@fgalz fgalz merged commit 0c916d2 into master Oct 19, 2024
1 check passed
@fgalz fgalz deleted the hotfix/temp-react-link-bug branch October 19, 2024 13:37
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.

1 participant