Skip to content

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

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

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

Conversation

@TheAnushree25

Copy link
Copy Markdown

🔍 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

netlify Bot commented Sep 12, 2025

Copy link
Copy Markdown

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
Copy Markdown
Author

@Altair-05 Please merge my PR

@Altair-05

Altair-05 commented Sep 12, 2025

Copy link
Copy Markdown
Owner

@Altair-05 Please merge my PR

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

@TheAnushree25

TheAnushree25 commented Sep 12, 2025

Copy link
Copy Markdown
Author

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

@Altair-05

Copy link
Copy Markdown
Owner

i want working vdo @TheAnushree25

@TheAnushree25

Copy link
Copy Markdown
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
Copy Markdown
Owner

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

@TheAnushree25

Copy link
Copy Markdown
Author

@Altair-05 is there any issue??

@TheAnushree25

Copy link
Copy Markdown
Author

@Altair-05 please tell

@TheAnushree25

Copy link
Copy Markdown
Author

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

@Altair-05 Altair-05 left a comment

Copy link
Copy Markdown
Owner

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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