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

Table, Multi-select: Add note to docs (HDS-4273) #2667

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

KristinLBradley
Copy link
Contributor

@KristinLBradley KristinLBradley commented Jan 24, 2025

📌 Summary

If merged, this PR updates the multi-select Table docs to add a note advising consumers to set an identityKey value in order to persist checkbox state.

👉 See related comment in PR to fix multi select Table bug.

Preview: https://hds-website-git-hds-4273-table-multi-select-do-e89064-hashicorp.vercel.app/components/table/table?tab=code#multi-select-table-using-a-model

📸 Screenshots

Added text is within the Info box

image

🔗 External links


👀 Component checklist

  • [ ] Percy was checked for any visual regression
  • [ ] A changelog entry was added via Changesets if needed (see templates here)

💬 Please consider using conventional comments when reviewing this PR.

@KristinLBradley KristinLBradley requested review from a team as code owners January 24, 2025 00:31
Copy link

vercel bot commented Jan 24, 2025

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

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Jan 24, 2025 5:51pm
hds-website ✅ Ready (Inspect) Visit Preview Jan 24, 2025 5:51pm

@hashibot-hds hashibot-hds added the docs-website Content updates to the documentation website label Jan 24, 2025
@MelSumner

This comment was marked as resolved.

didoo
didoo previously approved these changes Jan 24, 2025
@KristinLBradley
Copy link
Contributor Author

KristinLBradley commented Jan 24, 2025

@didoo Question: With this update to the docs, what should I do with the broader Jira ticket? Leave it open or set it to blocked or...?

@didoo
Copy link
Contributor

didoo commented Jan 24, 2025

@didoo Question: With this update to the docs, what should I do with the broader Jira ticket? Leave it open or set it to blocked or...?

Probably "Needs more info"?

@KristinLBradley KristinLBradley merged commit 2043d80 into main Jan 24, 2025
10 checks passed
@KristinLBradley KristinLBradley deleted the hds-4273-table-multi-select-docs-update branch January 24, 2025 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-website Content updates to the documentation website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants