Skip to content

Commit

Permalink
Task/wp 862 edit modal (#424)
Browse files Browse the repository at this point in the history
* Fix: Approved Expiration date field in edit modal does not match the input.

* Reverting some changes to test Sophia Massie-Pérez fix related to this fields

* Fix: Approved Expiration date discrepancy

* String to Date formating, refactor

* Update method name

---------

Co-authored-by: Chandra Y <[email protected]>
  • Loading branch information
jmartinez-tacc and chandra-tacc authored Feb 14, 2025
1 parent 21da26d commit bb177c5
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import Paginator from 'core-components/Paginator';
import styles from './ExtensionList.module.css';
import ViewExtensionModal from 'apcd-components/Extensions/ViewExtensionModal/ViewExtensionModal';
import EditExtensionModal from 'apcd-components/Extensions/EditExtensionModal/EditExtensionModal';
import { formatDate } from 'utils/dateUtil';
import { formatDate, formatUTCDate } from 'utils/dateUtil';
import { ClearOptionsButton } from 'apcd-components/ClearOptionsButton';

export const AdminExtensions: React.FC = () => {
Expand Down Expand Up @@ -134,7 +134,7 @@ export const AdminExtensions: React.FC = () => {
<td>{row.type}</td>
<td>{row.ext_outcome}</td>
<td>{row.ext_status}</td>
<td>{formatDate(row.approved_expiration_date)}</td>
<td>{formatUTCDate(row.approved_expiration_date)}</td>
<td className="modal-cell">
<select
id={`actionsDropdown_${row.ext_id}`}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import React from 'react';
import { Modal, ModalHeader, ModalBody, Row, Col } from 'reactstrap';
import { ExceptionRow } from 'hooks/admin';
import { formatModalDate } from 'utils/dateUtil';
import { formatUTCDate } from 'utils/dateUtil';
import styles from './ViewExceptionModal.module.css';

export const ViewExceptionModal: React.FC<{
Expand Down Expand Up @@ -50,7 +50,7 @@ export const ViewExceptionModal: React.FC<{
<Row>
<Col md={{ size: 4, offset: 1 }}>Created</Col>
<Col md={7}>
{(created_at && formatModalDate(created_at)) || 'None'}
{(created_at && formatUTCDate(created_at)) || 'None'}
</Col>
</Row>
<Row>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React, { useState, useEffect } from 'react';
import { Modal, ModalBody, ModalHeader, Row, Col, Alert } from 'reactstrap';
import { Field, useFormik, FormikHelpers, FormikProvider } from 'formik';
import { fetchUtil } from 'utils/fetchUtil';
import { formatDate } from 'utils/dateUtil';
import { formatDate, formatUTCDate } from 'utils/dateUtil';
import * as Yup from 'yup';
import { ExtensionRow } from 'hooks/admin';
import { useSubmitterDataPeriods } from 'hooks/entities';
Expand Down Expand Up @@ -115,8 +115,9 @@ const EditExtensionModal: React.FC<EditExtensionModalProps> = ({
value:
currentExtension?.approved_expiration_date &&
currentExtension?.approved_expiration_date !== 'None'
? formatDate(currentExtension?.requested_target_date)
: 'None',
? formatUTCDate(currentExtension?.approved_expiration_date)
: 'None'
+ 'currentExtension?.approved_expiration_date',
},
{
label: 'Extension Justification',
Expand Down
18 changes: 13 additions & 5 deletions apcd_cms/src/client/src/utils/dateUtil.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,19 @@ export const formatDate = (dateString: string | number | Date): string => {
}).format(date);
};

export const formatModalDate = (dateString: string | number | Date): string => {
return formatDate(dateString)
.replace('.', '')
.replace('AM', 'a.m.')
.replace('PM', 'p.m.');
export const formatUTCDate = (dateString: string | number | Date): string => {
const date = new Date(dateString);

if (isNaN(date.getTime())) {
return '';
}

return new Intl.DateTimeFormat('en-US', {
month: 'short',
day: 'numeric',
year: 'numeric',
timeZone: 'UTC'
}).format(date);
};

/**
Expand Down

0 comments on commit bb177c5

Please sign in to comment.