Skip to content

Commit 9ee0455

Browse files
authored
Merge pull request #1432 from Northeastern-Electric-Racing/#1279-only-show-allowed-expense-types
#1279: Filtered out expense types to only include allowed ones.
2 parents 101fe8a + ee56be6 commit 9ee0455

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

src/frontend/src/pages/FinancePage/ReimbursementRequestForm/ReimbursementFormView.tsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -194,11 +194,13 @@ const ReimbursementRequestFormView: React.FC<ReimbursementRequestFormViewProps>
194194
value={value}
195195
error={!!errors.expenseTypeId}
196196
>
197-
{allExpenseTypes.map((expenseType) => (
198-
<MenuItem key={expenseType.expenseTypeId} value={expenseType.expenseTypeId}>
199-
{expenseTypePipe(expenseType)}
200-
</MenuItem>
201-
))}
197+
{allExpenseTypes
198+
.filter((expenseType) => expenseType.allowed)
199+
.map((expenseType) => (
200+
<MenuItem key={expenseType.expenseTypeId} value={expenseType.expenseTypeId}>
201+
{expenseTypePipe(expenseType)}
202+
</MenuItem>
203+
))}
202204
</Select>
203205
)}
204206
/>

0 commit comments

Comments
 (0)