-
Notifications
You must be signed in to change notification settings - Fork 768
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
Add multi-select filter in organizations page for Headcount #1433
Conversation
const filter = { | ||
size: { eq: value }, | ||
or: value.map((count) => ({ size: count })), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here it needs to be:
or: value.map((count) => ({ size: { eq: count } })),
Otherwise API returns 500
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
{ | ||
label: '> 1000', | ||
value: '1000+', | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue scoping was not completely ok.
This specific attribute is following the canonical values from https://docs.peopledatalabs.com/docs/company-sizes
Instead of this new selection item, can you add support for
1001-5000
5001-10000
10001+
instead?
Apologies for the confusion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Changes proposed ✍️
What
Fixes #1242
🤖 Generated by Copilot at 3550e33
The headcount filter for organizations was improved to allow more flexibility and granularity. Users can now select multiple headcount ranges, including a new option for
>1000
, and the filter config and options were modified to support this feature.
🤖 Generated by Copilot at 3550e33
Why
How
🤖 Generated by Copilot at 3550e33
Checklist ✅
Feature
,Improvement
, orBug
.