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

feat: implement search feature for collection dropdown (#1130) #2038

Closed

Conversation

devin-ai-integration[bot]
Copy link
Contributor

This PR implements a searchable collection dropdown for the database policy configuration, addressing issue #1130.

Changes:

  • Created new SearchableSelect component with client-side filtering
  • Added translations for search placeholder and no-results message
  • Replaced existing Select component in AddRulesModal with SearchableSelect
  • Added proper form integration and error handling

Testing Status:

  • ✓ Lint checks passed
  • 🔄 Build verification delegated to CI pipeline

Link to Devin run: https://app.devin.ai/sessions/78b8f6da9f8541878f8f5fd27821edcc

Co-Authored-By: maslow@sealos.io <maslow@sealos.io>
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link

cla-assistant bot commented Jan 2, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

github-actions bot commented Jan 2, 2025

💥 laf web image built successfully for pr 2038

ghcr.io/labring/laf-web:pr-2038
docker.io/lafyun/laf-web:pr-2038

digest: sha256:e09b81560da0ccc1a90a552e50eaaac7aace2e021bf240bcd19acf4e87ed6061

Copy link
Contributor Author

Closing due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants