Skip to content

Conversation

@cappslock
Copy link
Contributor

@cappslock cappslock commented Dec 16, 2025

Adds EmptyState docs. Partially resolves https://github.com/shop/issues-retail/issues/21717.

This includes some unrelated changes to tab due to upstream changes in the definition file. It's safer to commit these than ignore them.

Copy link
Contributor

BrittaEvansFenton commented Dec 16, 2025

Did you mean to add the Tab stuff in here too? There is just no mention of it in the description. Please either remove or add into Descriptions

@cappslock
Copy link
Contributor Author

@BrittaEvansFenton it's generated output that's true to the source, so no, not intentional, but would also require just spinning up a second PR with those changes, which would block this PR. I'd honestly rather just get them in at once given the timeframe we're dealing with.

@BrittaEvansFenton
Copy link
Contributor

Absolutely, just edit the description so at least when we go back to figure out what we did later we have some context 😅

@cappslock cappslock merged commit 9be1dc6 into 2026-01-rc Dec 16, 2025
5 checks passed
@cappslock cappslock deleted the cappslock/chore/2026-01-rc/add-emptystate-docs branch December 16, 2025 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants