RyderTech/rsvp-manager
1
0
Fork 0
Code Issues Pull Requests Packages Activity

Fix type guard for filtering 'Other Items' to support string | undefined

Browse Source
This commit is contained in:
Ryderjj89
2025-05-04 17:17:16 -04:00
parent 40e84af553
commit 455dbf7ee6
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
frontend/src/components/EventAdmin.tsx
View File

@@ -772,7 +772,7 @@ const EventAdmin: React.FC = () => {
.map(r => r.other_items) .map(r => r.other_items)
.filter(Boolean) .filter(Boolean)
.flat() .flat()
.filter((item: string) => item && item.trim() !== ''); .filter((item): item is string => typeof item === 'string' && item.trim() !== '');
return allOtherItems.length > 0 return allOtherItems.length > 0
? allOtherItems.join(', ') ? allOtherItems.join(', ')
: 'No other items have been brought'; : 'No other items have been brought';

2
frontend/src/components/EventView.tsx
View File

@@ -249,7 +249,7 @@ const EventView: React.FC = () => {
.map(r => r.other_items) .map(r => r.other_items)
.filter(Boolean) .filter(Boolean)
.flat() .flat()
.filter((item: string) => item && item.trim() !== ''); .filter((item): item is string => typeof item === 'string' && item.trim() !== '');
return allOtherItems.length > 0 return allOtherItems.length > 0
? allOtherItems.join(', ') ? allOtherItems.join(', ')
: 'No other items have been brought'; : 'No other items have been brought';