Skip to content

feat(ENG-10767): filter integrations#7

Merged
adefreitas merged 5 commits into
mainfrom
ENG-10767/filter-integrations
Aug 19, 2025
Merged

feat(ENG-10767): filter integrations#7
adefreitas merged 5 commits into
mainfrom
ENG-10767/filter-integrations

Conversation

@adefreitas

@adefreitas adefreitas commented Aug 13, 2025

Copy link
Copy Markdown
Collaborator

Proceeded by https://github.com/StackOneHQ/malachite/pull/8

Implements integration search and category filters

Screen.Recording.2025-08-13.at.17.27.51.mov

Summary by cubic

Added search and category filters to the integration picker, making it easier to find and select integrations in the hub as required by ENG-10767.

  • New Features
  • Search bar for integrations.
  • Category filter using pill buttons.
  • Updated UI to show filtered results and empty states.

Copilot AI review requested due to automatic review settings August 13, 2025 16:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements search and category filtering functionality for the integration picker module. The changes enhance the user experience by allowing users to search for specific integrations by name and filter by category type.

Key changes:

  • Added search input and category filter pill buttons to the integration list
  • Refactored the CardFooter component to use showActions prop instead of isLoading
  • Added a resetConnectionState function to clear connection state when navigating back

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/modules/integration-picker/hooks/useIntegrationPicker.ts Added resetConnectionState callback function to reset connection state
src/modules/integration-picker/components/cardFooter.tsx Refactored prop from isLoading to showActions and adjusted padding
src/modules/integration-picker/components/IntegrationList.tsx Added search input, category filters, and conditional rendering for empty states
src/modules/integration-picker/components/IntegrationFields.tsx Added padding wrapper and fixed indentation
src/modules/integration-picker/IntegrationPicker.tsx Updated to use new CardFooter props and resetConnectionState function

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Comment thread src/modules/integration-picker/components/IntegrationList.tsx Outdated
Comment thread src/modules/integration-picker/components/IntegrationList.tsx Outdated
Comment thread src/modules/integration-picker/components/IntegrationFields.tsx
@adefreitas adefreitas changed the title [ENG-10767] feat: filter integrations feat(ENG-10767): filter integrations Aug 13, 2025
@adefreitas adefreitas requested a review from jpmiguel August 13, 2025 16:36

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

cubic analysis

No issues found across 5 files. Review in cubic

Linked issue analysis

Linked issue: ENG-10767: Search & filter

Status Acceptance criteria Notes
Add search in the new hub Search input and filtering logic added in IntegrationList.tsx
Category filter using pill buttons Category pills filter added with toggle logic in IntegrationList.tsx
Updated UI to show filtered results and empty states UI shows filtered integrations and 'No integrations found' message
Reset connection state on back actions resetConnectionState called on back button in IntegrationPicker.tsx

@adefreitas

Copy link
Copy Markdown
Collaborator Author

Missing a package version bump

@adefreitas

Copy link
Copy Markdown
Collaborator Author

Package version bumped

Comment thread src/modules/integration-picker/IntegrationPicker.tsx Outdated
@adefreitas adefreitas merged commit 05d0dc6 into main Aug 19, 2025
2 checks passed
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