Small improvements with views (#16821)
- when a view is deleted, it is removed from left menu + from list views in dropdown - deactivated fields are not suggested as options to group records by for a view (only in FE, not forbidden by BE)
This commit is contained in:
+8
@@ -6,6 +6,7 @@ import { getSnapshotValue } from '@/ui/utilities/state/utils/getSnapshotValue';
|
|||||||
import { usePersistView } from '@/views/hooks/internal/usePersistView';
|
import { usePersistView } from '@/views/hooks/internal/usePersistView';
|
||||||
import { useChangeView } from '@/views/hooks/useChangeView';
|
import { useChangeView } from '@/views/hooks/useChangeView';
|
||||||
import { useGetCurrentViewOnly } from '@/views/hooks/useGetCurrentViewOnly';
|
import { useGetCurrentViewOnly } from '@/views/hooks/useGetCurrentViewOnly';
|
||||||
|
import { coreViewsByObjectMetadataIdFamilySelector } from '@/views/states/selectors/coreViewsByObjectMetadataIdFamilySelector';
|
||||||
import { coreViewsFromObjectMetadataItemFamilySelector } from '@/views/states/selectors/coreViewsFromObjectMetadataItemFamilySelector';
|
import { coreViewsFromObjectMetadataItemFamilySelector } from '@/views/states/selectors/coreViewsFromObjectMetadataItemFamilySelector';
|
||||||
import { useCloseAndResetViewPicker } from '@/views/view-picker/hooks/useCloseAndResetViewPicker';
|
import { useCloseAndResetViewPicker } from '@/views/view-picker/hooks/useCloseAndResetViewPicker';
|
||||||
import { viewPickerIsDirtyComponentState } from '@/views/view-picker/states/viewPickerIsDirtyComponentState';
|
import { viewPickerIsDirtyComponentState } from '@/views/view-picker/states/viewPickerIsDirtyComponentState';
|
||||||
@@ -67,11 +68,18 @@ export const useDeleteViewFromCurrentState = (viewBarInstanceId?: string) => {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
set(
|
||||||
|
coreViewsByObjectMetadataIdFamilySelector(objectMetadataItem.id),
|
||||||
|
(views) =>
|
||||||
|
views.filter((view) => view.id !== viewPickerReferenceViewId),
|
||||||
|
);
|
||||||
|
|
||||||
await deleteView({ id: viewPickerReferenceViewId });
|
await deleteView({ id: viewPickerReferenceViewId });
|
||||||
},
|
},
|
||||||
[
|
[
|
||||||
currentView,
|
currentView,
|
||||||
closeAndResetViewPicker,
|
closeAndResetViewPicker,
|
||||||
|
objectMetadataItem.id,
|
||||||
changeView,
|
changeView,
|
||||||
deleteView,
|
deleteView,
|
||||||
viewPickerIsDirtyCallbackState,
|
viewPickerIsDirtyCallbackState,
|
||||||
|
|||||||
+2
-1
@@ -27,7 +27,8 @@ export const useGetAvailableFieldsForKanban = () => {
|
|||||||
|
|
||||||
const availableFieldsForKanban =
|
const availableFieldsForKanban =
|
||||||
objectMetadataItem?.readableFields.filter(
|
objectMetadataItem?.readableFields.filter(
|
||||||
(field) => field.type === FieldMetadataType.SELECT,
|
(field) =>
|
||||||
|
field.type === FieldMetadataType.SELECT && field.isActive === true,
|
||||||
) ?? [];
|
) ?? [];
|
||||||
|
|
||||||
const navigate = useNavigateSettings();
|
const navigate = useNavigateSettings();
|
||||||
|
|||||||
Reference in New Issue
Block a user