Skip to content

Conversation

@SupakornNetsuwan
Copy link
Collaborator

…text` search value.

Why did you create this PR

  • CollectionListViewProvider is redundant after custom-list-page render the ListView.
  • useTableStatesContext return the wrong search value, it must be debouncedSearch.
  • useSearch used to explicit return the internal search value state, now it returns props.search, props.onSearch if user provided.

What did you do

  • remove redundant CollectionListViewProvider
  • useTableStatesContext return correct search valie
  • useSearch return the user defined controlled value
  • renamed pages to page in list UI config

Checklist

  • Self-reviewed your code
  • Wrote coverage tests
  • Added screenshots or recordings if applicable

@changeset-bot
Copy link

changeset-bot bot commented Aug 26, 2025

🦋 Changeset detected

Latest commit: 2b09426

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

This PR includes changesets to release 8 packages
Name Type
@example/erp Patch
@genseki/react Patch
@example/ui-playground Patch
@genseki/next Patch
@genseki/plugins Patch
@genseki/prisma-generator Patch
@genseki/react-query Patch
@genseki/rest 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

@saenyakorn
Copy link
Member

Please change the head to #162

@SupakornNetsuwan SupakornNetsuwan changed the base branch from main to yu/feat/refactor-collection August 26, 2025 08:55
Base automatically changed from yu/feat/refactor-collection to main August 27, 2025 05:40
@saenyakorn
Copy link
Member

Fixed by #162

@saenyakorn saenyakorn closed this Aug 27, 2025
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