Skip to content

DOP-6429: Fixing dissmissable skills card bug#1601

Merged
biancalaube merged 4 commits intomainfrom
DOP-6429
Feb 13, 2026
Merged

DOP-6429: Fixing dissmissable skills card bug#1601
biancalaube merged 4 commits intomainfrom
DOP-6429

Conversation

@biancalaube
Copy link
Copy Markdown
Collaborator

@biancalaube biancalaube commented Feb 10, 2026

Stories/Links:

DOP-6429

Bug in Dismissible skills card not dismissing

Current Behavior:

https://www.mongodb.com/docs/manual/data-modeling/

Staging Links:

https://deploy-preview-1601--docs-frontend-stg.netlify.app/docs/manual/data-modeling/

Notes:

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

@docs-builder-bot
Copy link
Copy Markdown

docs-builder-bot commented Feb 10, 2026

✅ Deploy Preview for atlas ready

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

@docs-builder-bot
Copy link
Copy Markdown

docs-builder-bot commented Feb 10, 2026

✅ Deploy Preview for manual ready

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

Copy link
Copy Markdown
Collaborator

@mmeigs mmeigs left a comment

Choose a reason for hiding this comment

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

Thank youuuu

Comment on lines 128 to 130
</Box>
<CloseButton onClick={() => onClose} />
<CloseButton onClick={() => onClose()} />
<Body>Master "{skill}" for free!</Body>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Wow, yep. Nailed it.
nit: IF you like, you can just do onClick={onClose} unless the LG types are odd

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

made this fix thanks Matt!

@biancalaube biancalaube merged commit 6799673 into main Feb 13, 2026
7 checks passed
@biancalaube biancalaube deleted the DOP-6429 branch February 13, 2026 16:33
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