Skip to content

Conversation

@seungpark
Copy link
Collaborator

@seungpark seungpark commented Sep 30, 2025

Stories/Links:

DOP-6197

Current Behavior:

This link contains a ref hash id (test in incognito window or clear local storage)
http://mongodb.com/docs/atlas/atlas-search/tutorial/#create-the--index.-18
This does not lead users to any significant portion of the page, just makes default selections.

Staging Links:

This staging link takes users to the correct selections and to the ref on the page:
https://deploy-preview-1563--docs-frontend-stg.netlify.app/docs/atlas/atlas-search/tutorial/#create-the--index.-18

Notes:

  • This behavior is done by memoizing all the hashes included in each composable content at build time, and verifying if the hash in URL belongs to a selected content.
  • In the case that multiple composable content have the same ref, this is a content issue and will result in the first composable content being returned.

README updates

    • This PR introduces changes that should be reflected in the README, and I have made those updates.
    • This PR does not introduce changes that should be reflected in the README

@netlify
Copy link

netlify bot commented Sep 30, 2025

Deploy Preview for docs-frontend-stg ready!

Name Link
🔨 Latest commit ecb4494
🔍 Latest deploy log https://app.netlify.com/projects/docs-frontend-stg/deploys/68dc4800e564340008e535ff
😎 Deploy Preview https://deploy-preview-1563--docs-frontend-stg.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 project configuration.

@docs-builder-bot
Copy link

docs-builder-bot commented Sep 30, 2025

✅ Deploy Preview for atlas ready

Item Details
🔨 Latest Commit 0de2b5d420dc1915f5784353afa6d5af2362d2d5
😎 Deploy Preview https://deploy-preview-1563--docs-frontend-stg.netlify.app/docs/atlas
🔍 Build Logs View Logs

@seungpark seungpark marked this pull request as ready for review September 30, 2025 19:02
Copy link
Collaborator

@biancalaube biancalaube left a comment

Choose a reason for hiding this comment

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

LGTM! one small NiT

@seungpark seungpark merged commit 11ed591 into main Sep 30, 2025
7 checks passed
@seungpark seungpark deleted the DOP-6197 branch September 30, 2025 21:28
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.

3 participants