Skip to content

Conversation

@sagardspeed2
Copy link
Contributor

@sagardspeed2 sagardspeed2 commented Jan 23, 2025

Describe your changes

Screenshots [Optional]

Issue ticket number and link

Closes #1006

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added tests
  • I have added a changeset pnpm changeset add
  • I have added example usage in the kitchen sink app

@changeset-bot
Copy link

changeset-bot bot commented Jan 23, 2025

🦋 Changeset detected

Latest commit: 028e0b0

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

This PR includes changesets to release 4 packages
Name Type
@ensembleui/react-kitchen-sink Patch
@ensembleui/react-runtime Patch
@ensembleui/react-preview Patch
@ensembleui/react-starter 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

@sagardspeed2 sagardspeed2 requested a review from evshi January 23, 2025 14:31
@sagardspeed2 sagardspeed2 self-assigned this Jan 23, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2025

Visit the preview URL for this PR (updated for commit 028e0b0):

https://react-kitchen-sink-dev--pr1008-1006-support-dynamic-bw9a0tyf.web.app

(expires Fri, 31 Jan 2025 20:25:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6267897ade2ba783b6db70a53a60fc3946d625e9

Copy link
Contributor

@evshi evshi left a comment

Choose a reason for hiding this comment

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

Needs test

Copy link
Contributor

@evshi evshi left a comment

Choose a reason for hiding this comment

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

can you add a test for a static search key also.

values?.searchKey
? [itemTemplate?.name ?? "", values.searchKey]
: [(itemTemplate?.value || itemTemplate?.name) ?? ""],
: itemTemplate?.value || itemTemplate?.name || "",
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean to remove this here?

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.

Can't pass searchKey a dynamic value

2 participants