Skip to content

Commit a721053

Browse files
authored
Merge pull request #2170 from Northeastern-Electric-Racing/#2081-RR-receipts-not-required-locally
#2081 Receipt Upload not Required in RR form locally
2 parents 1aae413 + da0ab98 commit a721053

1 file changed

Lines changed: 13 additions & 5 deletions

File tree

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

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,8 @@ interface ReimbursementRequestFormProps {
4848
previousPage: string;
4949
}
5050

51+
const RECEIPTS_REQUIRED = import.meta.env.VITE_RR_RECEIPT_REQUIREMENT || 'disabled';
52+
5153
const schema = yup.object().shape({
5254
vendorId: yup.string().required('Vendor is required'),
5355
account: yup.string().required('Account is required'),
@@ -67,11 +69,17 @@ const schema = yup.object().shape({
6769
)
6870
.required('reimbursement products required')
6971
.min(1, 'At least one Reimbursement Product is required'),
70-
receiptFiles: yup
71-
.array()
72-
.required('receipt files required')
73-
.min(1, 'At least one Receipt is required')
74-
.max(7, 'At most 7 Receipts are allowed')
72+
receiptFiles:
73+
// The requirements for receipt uploads is disabled by default on development to make testing easier;
74+
// if testing proper receipt uploads is needed, create an environment variable called VITE_RR_RECEIPT_REQUIREMENT
75+
// in src/frontend/.env and set it to 'enabled'.
76+
import.meta.env.MODE === 'development' && RECEIPTS_REQUIRED !== 'enabled'
77+
? yup.array()
78+
: yup
79+
.array()
80+
.required('receipt files required')
81+
.min(1, 'At least one Receipt is required')
82+
.max(7, 'At most 7 Receipts are allowed')
7583
});
7684

7785
const ReimbursementRequestForm: React.FC<ReimbursementRequestFormProps> = ({

0 commit comments

Comments
 (0)