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

DOP-5290: fix links destinations #67

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

DOP-5290: fix links destinations #67

wants to merge 3 commits into from

Conversation

seungpark
Copy link
Collaborator

@seungpark seungpark commented Jan 22, 2025

DOP-5290

This PR aims to fix broken href links in the offline docs. Previous iteration was reading node.href when scanning the DOM tree for anchor tags, which would include prefixes like file:// or about://. This iteration uses getAttribute to fetch the correct href/src value and replace it with a relative path.

Successful deploy logs here

The uploaded offline doc can be found at this link: https://us-east-2.console.aws.amazon.com/s3/buckets/docs-mongodb-org-prd?region=us-east-2&bucketType=general&prefix=docs/offline/&showversions=false

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for redoc-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/redoc-extension/deploys/67914c2d42ac880007c17ac3

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for snooty-cache-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/snooty-cache-extension/deploys/67914c2d71375b00088728c3

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for populate-data-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/populate-data-extension/deploys/67914c2d1739630008ebac01

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for search-manifest-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/search-manifest-extension/deploys/67914c2d71375b00088728c5

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for git-changed-file-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/git-changed-file-extension/deploys/67914c2dc20efc00082059f0

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for persistence-module-ext canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/persistence-module-ext/deploys/67914c2da8270200088cf9f9

Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for redirects-and-publish-extension canceled.

Name Link
🔨 Latest commit 6201b03
🔍 Latest deploy log https://app.netlify.com/sites/redirects-and-publish-extension/deploys/67914c2d0bbf8200082f8fa1

console.log('Skipping offline extension for env ', environment.env);
return;
}
// if (!ENVS_TO_RUN.includes(environment.ENV as string)) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uncomment before merge. or can handle as part of DOP-5294

@seungpark seungpark marked this pull request as ready for review January 23, 2025 14:02
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