Skip to content

Commit ae1be71

Browse files
committed
Merge branch 'filip-refactor-users-table' into filip-dry-subscription-status
2 parents c0675b7 + 91fe98c commit ae1be71

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

template/app/src/admin/dashboards/users/UsersTable.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function AdminSwitch({ id, isAdmin }: Pick<User, 'id' | 'isAdmin'>) {
1515

1616
const UsersTable = () => {
1717
const [currentPage, setCurrentPage] = useState(1);
18-
const [emailFilter, setEmailFilter] = useState<string>('');
18+
const [emailFilter, setEmailFilter] = useState<string | undefined>('');
1919
const [isAdminFilter, setIsAdminFilter] = useState<boolean | undefined>(undefined);
2020
const [subscriptionStatusFilter, setSubcriptionStatusFilter] = useState<Array<SubscriptionStatus | null>>(
2121
[]
@@ -28,7 +28,7 @@ const UsersTable = () => {
2828
filter: {
2929
...(emailFilter && { emailContains: emailFilter }),
3030
...(isAdminFilter !== undefined && { isAdmin: isAdminFilter }),
31-
...(subscriptionStatusFilter?.length > 0 && { subscriptionStatusIn: subscriptionStatusFilter }),
31+
...(subscriptionStatusFilter.length > 0 && { subscriptionStatusIn: subscriptionStatusFilter }),
3232
},
3333
});
3434

@@ -54,7 +54,8 @@ const UsersTable = () => {
5454
id='email-filter'
5555
placeholder='[email protected]'
5656
onChange={(e) => {
57-
setEmailFilter(e.currentTarget.value);
57+
const value = e.currentTarget.value;
58+
setEmailFilter(value === '' ? undefined : value);
5859
}}
5960
className='rounded border border-stroke py-2 px-5 bg-white outline-none transition focus:border-primary active:border-primary disabled:cursor-default disabled:bg-whiter dark:border-form-strokedark dark:bg-form-input dark:focus:border-primary'
6061
/>
@@ -63,7 +64,7 @@ const UsersTable = () => {
6364
</label>
6465
<div className='flex-grow relative z-20 rounded border border-stroke pr-8 outline-none bg-white transition focus:border-primary active:border-primary dark:border-form-strokedark dark:bg-form-input'>
6566
<div className='flex items-center'>
66-
{!!subscriptionStatusFilter && subscriptionStatusFilter.length > 0 ? (
67+
{subscriptionStatusFilter.length > 0 ? (
6768
subscriptionStatusFilter.map((opt) => (
6869
<span
6970
key={opt}

0 commit comments

Comments
 (0)