Skip to content

Commit f285150

Browse files
committed
removed redundant sabo id formatter
1 parent 2ed9631 commit f285150

5 files changed

Lines changed: 6 additions & 18 deletions

File tree

src/frontend/src/pages/FinancePage/FinanceComponents/ReimbursementRequestInfo.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import {
99
centsToDollar,
1010
datePipe,
1111
dateUndefinedPipe,
12-
formatSaboIdPipe
1312
} from '../../../utils/pipes';
1413
import {
1514
createReimbursementRequestRowData,
@@ -161,7 +160,7 @@ const ReimbursementRequestInfo = ({
161160
headerName: 'SABO ID',
162161
flex: 0.5,
163162
minWidth: 100,
164-
valueGetter: (params: any) => formatSaboIdPipe(params.row.saboId)
163+
valueGetter: (params: any) => params.row.saboId
165164
},
166165
{
167166
field: 'dateSubmitted',

src/frontend/src/pages/FinancePage/ReimbursementRequestDetailPage/ReimbursementRequestDetailsView.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* See the LICENSE file in the repository root folder for details.
44
*/
55

6-
import { accountCodePipe, displayEnum, formatSaboIdPipe } from '../../../utils/pipes';
6+
import { accountCodePipe, displayEnum } from '../../../utils/pipes';
77
import { Assignment, ChangeCircle, Edit, Pending } from '@mui/icons-material';
88
import CheckIcon from '@mui/icons-material/Check';
99
import CloseIcon from '@mui/icons-material/Close';
@@ -465,7 +465,7 @@ const ReimbursementRequestDetailsView: React.FC<ReimbursementRequestDetailsViewP
465465
},
466466
{ content: `$${centsToDollar(reimbursementRequest.totalCost)}` },
467467
{ content: reimbursementRequest.vendor.name },
468-
{ content: `${formatSaboIdPipe(reimbursementRequest.saboId)}` },
468+
{ content: `${reimbursementRequest.saboId}` },
469469
{
470470
content: refundSourceNames.join(', ')
471471
},

src/frontend/src/utils/pipes.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -210,14 +210,3 @@ export const labelPipe = (label: string) => {
210210

211211
return result;
212212
};
213-
214-
// Pad SABO ID with leading zeroes
215-
export const formatSaboIdPipe = (saboId: number | undefined): string => {
216-
if (saboId === undefined || saboId === null) return undefinedPipe(saboId as any);
217-
const str = String(saboId);
218-
// Only pad if it's shorter than 5
219-
if (str.length < 5) {
220-
return str.padStart(5, '0');
221-
}
222-
return str;
223-
};

src/shared/src/types/finance-types.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ export interface ReimbursementRequestData {
8888
totalBudget: number;
8989
approved: number;
9090
pendingApproval: number;
91-
addedToSabo: number;
91+
addedToSabo: string;
9292
reimbursed: number;
9393
available: number;
9494
}

src/shared/src/types/reimbursement-requests-types.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ export interface IndexCode {
1212
export interface ReimbursementRequestRow {
1313
identifier: number;
1414
id: string;
15-
saboId: number | undefined;
15+
saboId: string | undefined;
1616
amount: number;
1717
dateSubmitted: Date;
1818
status: ReimbursementStatusType;
@@ -51,7 +51,7 @@ export interface Receipt {
5151
export interface ReimbursementRequest {
5252
reimbursementRequestId: string;
5353
identifier: number;
54-
saboId?: number;
54+
saboId?: string;
5555
dateCreated: Date;
5656
dateOfExpense?: Date;
5757
reimbursementStatuses: ReimbursementStatus[];

0 commit comments

Comments
 (0)