tabshift/payloadcms
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

Merge pull request #1766 from jacobsfletch/fix/1765

Browse Source 
fix: error clearing date field
This commit is contained in:
James Mikrut
2023-01-02 09:51:51 -05:00
committed by GitHub
parent d919abbda4 420ed6738e
commit e330d8874b
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/admin/components/forms/field-types/DateTime/index.tsx
View File

@@ -89,7 +89,7 @@ const DateTime: React.FC<Props> = (props) => {
placeholder={getTranslation(placeholder, i18n)}
readOnly={readOnly}
onChange={(incomingDate) => {
if (!readOnly) setValue(incomingDate.toISOString());
if (!readOnly) setValue(incomingDate?.toISOString() || null);
}}
value={value as Date}
/>

12
test/fields/e2e.spec.ts
View File

@@ -374,6 +374,18 @@ describe('fields', () => {
url = new AdminUrlUtil(serverURL, 'date-fields');
});
test('should clear date', async () => {
await page.goto(url.create);
const dateField = page.locator('#field-default input');
await expect(dateField).toBeVisible();
await dateField.fill('2021-08-01');
await expect(dateField).toHaveValue('2021-08-01');
const clearButton = page.locator('#field-default .date-time-picker__clear-button');
await expect(clearButton).toBeVisible();
await clearButton.click();
await expect(dateField).toHaveValue('');
});
test('should display formatted date in list view if displayFormat option added to date field', async () => {
await page.goto(url.list);
const formattedDateCell = page.locator('.row-1 .cell-timeOnly');