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

Collection view: table columns definition #1249

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Conversation

jossmac
Copy link
Member

@jossmac jossmac commented Jul 30, 2024

Implements the columns definition proposal, while maintaining the original API.

Thank you jplhomer for your work in #1110, we appreciate your patience on this feature.

Resolves #1250

Copy link

changeset-bot bot commented Jul 30, 2024

🦋 Changeset detected

Latest commit: eb3fed1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages
Name Type
@keystatic/core Patch
keystatic-docs Patch
@keystar/docs Patch
@example/astro-content Patch
@example/astro Patch
@example/localization Patch
@example/next-app Patch
@example/next-block-builder Patch
@example/next-pages Patch
@keystatic/remix-test-app Patch
@keystatic/templates-astro Patch
@keystatic/templates-nextjs Patch
@keystatic/templates-remix Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@@ -37,29 +37,84 @@ export default config({

### Columns

`columns` — show additional fields in the collection list view.
By default, only the “slug” of each entry is displayed in the collection table. You can show additional fields by passing a `columns` option.
Copy link
Member

Choose a reason for hiding this comment

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

By default, only the “slug” of each entry is displayed in the collection table. You can show additional fields by passing a columns option.

This is no longer true with this PR, columns now removes the slug instead of just adding other columns. I don't think we should land this without preserving the ability to show the slug and other fields.

@jossmac jossmac marked this pull request as draft July 31, 2024 01:35
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.

Columns definition
2 participants