Skip to content

feat: Add search and filter options to Customers page#72

Open
TheAnushree25 wants to merge 1 commit intoAltair-05:mainfrom
TheAnushree25:feature/customers-search-filter
Open

feat: Add search and filter options to Customers page#72
TheAnushree25 wants to merge 1 commit intoAltair-05:mainfrom
TheAnushree25:feature/customers-search-filter

Conversation

@TheAnushree25
Copy link

🔍 Problem
The Customers page previously displayed all customer logos in one grid, with no search or filter options. As the customer list grew, navigation became harder and inefficient.

💡 Solution
🔎 Search Bar – quickly find customers by name.
🏷️ Category/Industry Filters – narrow down results by category.
📦 Refactored Data – moved customer data into a JSON/array structure for easier filtering and future scalability.
🛠️ Implementation

Created a structured customersData array with { id, name, category, logo }.
Added real-time search functionality using useState and controlled input.
Added filter dropdown to select categories/industries.
Updated UI grid to show only matching customers.

✅ Testing
Searched by customer name → results update instantly.
Selected a category → only relevant customers appear.
Combined search + filter → works as expected.
Checked edge cases (empty search, no matches).

@netlify
Copy link

netlify bot commented Sep 12, 2025

Deploy Preview for codebrosplatform ready!

Name Link
🔨 Latest commit 159f914
🔍 Latest deploy log https://app.netlify.com/projects/codebrosplatform/deploys/68c3c68d92c8ca0009dc5630
😎 Deploy Preview https://deploy-preview-72--codebrosplatform.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@TheAnushree25
Copy link
Author

@Altair-05 Please merge my PR

@Altair-05
Copy link
Owner

Altair-05 commented Sep 12, 2025

@Altair-05 Please merge my PR

@TheAnushree25 could u plz attach images and working vdo of chnages u made

@TheAnushree25
Copy link
Author

TheAnushree25 commented Sep 12, 2025

@Altair-05 Screenshot 2025-09-12 150752
Screenshot 2025-09-12 150800
Added search bar and features for customer filtering

@Altair-05
Copy link
Owner

i want working vdo @TheAnushree25

@TheAnushree25
Copy link
Author

@Altair-05

localhost_5181_customers.-.Google.Chrome.2025-09-12.15-39-57.mp4

ma'am this is the working video .....please merge my PR

@Altair-05
Copy link
Owner

@TheAnushree25 it will be reviewed and merged once it's ready.

@TheAnushree25
Copy link
Author

@Altair-05 is there any issue??

@TheAnushree25
Copy link
Author

@Altair-05 please tell

@TheAnushree25
Copy link
Author

@Altair-05 ma'am please merge it .....it took a lot of efforts to resolve the issue..........

Copy link
Owner

@Altair-05 Altair-05 left a comment

Choose a reason for hiding this comment

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

The PR deletes all network page UI, which is unrelated and a breaking change for another feature.

Copy link
Owner

Choose a reason for hiding this comment

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

Duplicates customer data logic
Uses hardcoded assets and array instead of a shared data source, which may cause future inconsistency.

Copy link
Owner

Choose a reason for hiding this comment

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

This file is never actually imported or used in customers.jsx hence dead code. If left unused, it is unnecessary/confusing.

Copy link
Owner

Choose a reason for hiding this comment

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

Also never imported/used. Duplicates the intent of customersData.jsx and the array in customers.jsx. This is now a third separate definition of customers, creating confusion.

Copy link
Owner

Choose a reason for hiding this comment

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

The main UI rendering for filters and developer grid is replaced with a placeholder comment {/* Your existing JSX for network page here */}. This removes all actual UI for the Network page, which is a breaking change. This is a critical regression and not related to the Customers feature.

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.

2 participants