Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #1376

Merged
merged 14 commits into from
Oct 1, 2024
Merged

Develop #1376

Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/components/RecurringEvents/RecurringEvents.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ const RecurringEvents = function ({
setFormFields,
dateType,
disabledDate,
onOpenChange,
onCalendarChange,
setSubEventCount,
subEventCount,
Expand Down Expand Up @@ -349,6 +350,7 @@ const RecurringEvents = function ({
style={{ width: '423px' }}
disabled={(isCustom || formFields?.frequency === 'CUSTOM') && startDateRecur?.length == 2 && true}
disabledDate={disabledDate}
onOpenChange={onOpenChange}
onCalendarChange={onCalendarChange}
suffixIcon={
subEventCount > 0 && (
Expand Down
75 changes: 54 additions & 21 deletions src/constants/formFields.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,17 +70,40 @@ export const formNames = {
const rules = [
{
dataType: dataTypes.URI_STRING,
rule: {
type: 'url',
message: <Translation>{(t) => t('dashboard.events.addEditEvent.validations.url')}</Translation>,
},
rule: () => [
{
type: 'url',
message: <Translation>{(t) => t('dashboard.events.addEditEvent.validations.url')}</Translation>,
},
],
},
{
dataType: dataTypes.EMAIL,
rule: {
type: 'email',
message: <Translation>{(t) => t('login.validations.invalidEmail')}</Translation>,
},
rule: () => [
{
type: 'email',
message: <Translation>{(t) => t('login.validations.invalidEmail')}</Translation>,
},
],
},
{
dataType: dataTypes.IMAGE,
rule: ({ image, t }) => [
({ getFieldValue }) => ({
validator() {
if (
(getFieldValue('image') != undefined && getFieldValue('image')?.length > 0) ||
(image && !getFieldValue('image')) ||
(image && getFieldValue('image')?.length > 0)
) {
return Promise.resolve();
} else
return Promise.reject(
new Error(t('dashboard.events.addEditEvent.validations.otherInformation.emptyImage')),
);
},
}),
],
},
];

Expand Down Expand Up @@ -115,7 +138,9 @@ export const formFieldValue = [
name={Array.isArray(name) ? name[0] : name}
data={data}
entityId={entityId}
validations={validations}
validations={
validations && validations.trim() !== '' ? validations : t('common.validations.informationRequired')
}
dataCy={`input-text-area-${mappedField}-`}
placeholder={placeholder}
required={required}>
Expand Down Expand Up @@ -347,7 +372,6 @@ export const formFieldValue = [
locationPlace,
setLocationPlace,
t,
name,
placesSearch,
calendarContentLanguage,
allPlacesArtsdataList,
Expand All @@ -359,7 +383,7 @@ export const formFieldValue = [
fieldName,
}) => {
return (
<>
<div>
<Popover
data-cy={`popover-${mappedField ?? fieldName}`}
open={isPopoverOpen}
Expand Down Expand Up @@ -395,7 +419,7 @@ export const formFieldValue = [
}`}
onClick={() => {
setLocationPlace(place);
form.setFieldValue(name, place?.value);
form.setFieldValue(fieldName, place?.value);
setIsPopoverOpen(false);
}}
data-cy={`div-${mappedField ?? fieldName}-footlight-place-${index}`}>
Expand Down Expand Up @@ -429,7 +453,7 @@ export const formFieldValue = [
}`}
onClick={() => {
setLocationPlace(place);
form.setFieldValue(name, place?.value);
form.setFieldValue(fieldName, place?.value);
setIsPopoverOpen(false);
}}
data-cy={`div-${mappedField ?? fieldName}-footlight-place-${index}`}>
Expand Down Expand Up @@ -460,7 +484,7 @@ export const formFieldValue = [
className="event-popover-options"
onClick={() => {
setLocationPlace(place);
form.setFieldValue(name, place?.uri);
form.setFieldValue(fieldName, place?.uri);
setIsPopoverOpen(false);
}}
data-cy={`div-${mappedField ?? fieldName}-artsdata-place-${index}`}>
Expand Down Expand Up @@ -501,14 +525,14 @@ export const formFieldValue = [
closable
onClose={() => {
setLocationPlace();
form.setFieldValue(name, undefined);
form.setFieldValue(fieldName, undefined);
}}
edit={locationPlace?.source === sourceOptions.CMS && true}
onEdit={(e) => placeNavigationHandler(locationPlace?.value, locationPlace?.type, e)}
creatorId={locationPlace?.creatorId}
/>
)}
</>
</div>
);
},
},
Expand Down Expand Up @@ -557,6 +581,7 @@ export const renderFormFields = ({
mappedField,
t,
validations,
originalUrl,
fieldName,
}) => {
return (
Expand All @@ -565,7 +590,7 @@ export const renderFormFields = ({
<Form.Item
data-cy={`form-item-${mappedField ?? fieldName?.toLowerCase()}`}
label={label}
name={name}
name={name ?? fieldName?.toLowerCase()}
key={key}
initialValue={
Array.isArray(initialValue)
Expand All @@ -582,14 +607,18 @@ export const renderFormFields = ({
className={mappedField ?? fieldName?.toLowerCase()}
rules={rules
?.map((rule) => {
if (datatype === rule?.dataType) return rule.rule;
if (datatype === rule?.dataType) {
return rule.rule({ image: originalUrl, t, required, fieldName });
}
})
.concat([
{
required: required,
message: validations ?? t('common.validations.informationRequired'),
message:
validations && validations.trim() !== '' ? validations : t('common.validations.informationRequired'),
},
])
?.flat()
?.filter((rule) => rule !== undefined)}
help={
position === 'bottom' && userTips ? (
Expand Down Expand Up @@ -641,7 +670,7 @@ export const returnFormDataWithFields = ({
}) => {
return renderFormFields({
fieldName: field?.name,
name: [field?.mappedField],
name: field?.mappedField && [field?.mappedField],
mappedField: field?.mappedField,
type: field?.type,
datatype: field?.datatype,
Expand All @@ -657,7 +686,7 @@ export const returnFormDataWithFields = ({
type: field?.type,
isDynamicField: false,
calendarContentLanguage,
name: [field?.mappedField],
name: field?.mappedField && [field?.mappedField],
preview: true,
placeholder: bilingual({
data: field?.placeholder,
Expand Down Expand Up @@ -737,5 +766,9 @@ export const returnFormDataWithFields = ({
interfaceLanguage: user?.interfaceLanguage?.toLowerCase(),
locationPlace,
}),
originalUrl:
field?.mappedField === mappedFieldTypes.IMAGE
? entityData?.image?.find((image) => image?.isMain)?.original?.uri
: field?.mappedField === mappedFieldTypes.LOGO && entityData?.logo?.original?.uri,
});
};
6 changes: 6 additions & 0 deletions src/pages/Dashboard/AddEvent/AddEvent.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -2701,6 +2701,9 @@ function AddEvent() {
setStartDate(dates?.[0]);
setEndDate(dates?.[1]);
}}
onOpenChange={(open) => {
if (!open && startDate && !endDate) setStartDate(null);
}}
disabledDate={(current) =>
(startDate && current.isSame(startDate, 'day')) ||
(endDate && current.isSame(endDate, 'day'))
Expand All @@ -2723,6 +2726,9 @@ function AddEvent() {
setStartDate(dates?.[0]);
setEndDate(dates?.[1]);
}}
onOpenChange={(open) => {
if (!open && startDate && !endDate) setStartDate(null);
}}
disabledDate={(current) =>
(startDate && current.isSame(startDate, 'day')) ||
(endDate && current.isSame(endDate, 'day'))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ function CreateNewOrganization() {
collection.push([field?.mappedField, contentLanguageKeyMap[language]]);
});
return collection;
} else return field?.mappedField;
} else return field?.mappedField ?? field?.name?.toLowerCase();
})
?.flat(),
);
Expand Down
2 changes: 1 addition & 1 deletion src/pages/Dashboard/Settings/Settings.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ const Settings = () => {
{
label: <span data-cy="tab-widget-settings">{t('dashboard.settings.tab2')}</span>,
key: '2',
children: <WidgetSettings tabKey={tabKey} setDirtyStatus={setIsFormDirty} />,
children: <WidgetSettings tabKey={tabKey} />,
disabled: false,
adminOnly: true,
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import { widgetFontCollection } from '../../../../constants/fonts';
const { useBreakpoint } = Grid;
const widgetUrl = process.env.REACT_APP_CALENDAR_WIDGET_BASE_URL;

const WidgetSettings = ({ setDirtyStatus, tabKey }) => {
const WidgetSettings = ({ tabKey }) => {
const { t } = useTranslation();
const { calendarId } = useParams();
const timestampRef = useRef(Date.now()).current;
Expand Down Expand Up @@ -194,7 +194,6 @@ const WidgetSettings = ({ setDirtyStatus, tabKey }) => {
setIframeCode(
`<iframe src="${urlCopy.href}" width="100%" style="max-width:${width}px; border:none" height="${height}px"></iframe>`,
);
setDirtyStatus(true);
}
};

Expand Down
21 changes: 14 additions & 7 deletions src/utils/dateTimeTypeHandler.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,18 @@ import { dateTypes } from '../constants/dateTypes';

export const dateTimeTypeHandler = (startDate, startDateTime, endDate, endDateTime, isRecurring) => {
if (isRecurring) return dateTypes.MULTIPLE;
else if ((startDate || startDateTime) && !endDate && !endDateTime) return dateTypes.SINGLE;
else if ((startDate || startDateTime) && endDateTime && !endDate) {
if (startDate && moment(startDate).isSame(endDateTime, 'day')) return dateTypes.SINGLE;
else if (startDate && !moment(startDate).isSame(endDateTime, 'day')) return dateTypes.RANGE;
else if (startDateTime && moment(startDateTime).isSame(endDateTime, 'day')) return dateTypes.SINGLE;
else if (startDateTime && !moment(startDateTime).isSame(endDateTime, 'day')) return dateTypes.RANGE;
} else if ((startDate || startDateTime) && !endDateTime && endDate) return dateTypes.RANGE;

const start = startDateTime || startDate;
const end = endDateTime || endDate;

// only start is provided
if (start && !end) return dateTypes.SINGLE;

// if both start and end are present
if (start && end) {
// If start and end are the same day, or within 24 hours
if (moment(start).isSame(end, 'day') || moment(end).diff(moment(start), 'hours') < 24) return dateTypes.SINGLE;

return dateTypes.RANGE;
}
};
Loading