From 48347095d2813d8a9d87494bb06745a1b8206278 Mon Sep 17 00:00:00 2001 From: Lucas Bordeau Date: Tue, 24 Jun 2025 13:57:18 +0200 Subject: [PATCH] Fixed ACTOR advanced filter recent regressions (#12813) This PR fixes recent regressions on advanced filters for the ACTOR field type. - The new `isFilterable` props on `SETTINGS_COMPOSITE_FIELD_TYPE_CONFIGS` wasn't taken into account for sub field picker in advanced filter. - A wrong component instance id was passed to `subFieldNameUsedInDropdownComponentState` --- .../components/AdvancedFilterDropdownFilterInput.tsx | 1 - .../components/AdvancedFilterSubFieldSelectMenu.tsx | 4 +++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterDropdownFilterInput.tsx b/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterDropdownFilterInput.tsx index 661a21991..da1e637ab 100644 --- a/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterDropdownFilterInput.tsx +++ b/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterDropdownFilterInput.tsx @@ -32,7 +32,6 @@ export const AdvancedFilterDropdownFilterInput = ({ }: AdvancedFilterDropdownFilterInputProps) => { const subFieldNameUsedInDropdown = useRecoilComponentValueV2( subFieldNameUsedInDropdownComponentState, - filterDropdownId, ); const filterType = recordFilter.type; diff --git a/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterSubFieldSelectMenu.tsx b/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterSubFieldSelectMenu.tsx index 6f65a0c42..96307995d 100644 --- a/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterSubFieldSelectMenu.tsx +++ b/packages/twenty-front/src/modules/object-record/advanced-filter/components/AdvancedFilterSubFieldSelectMenu.tsx @@ -94,7 +94,9 @@ export const AdvancedFilterSubFieldSelectMenu = ({ const subFieldNames = SETTINGS_COMPOSITE_FIELD_TYPE_CONFIGS[ objectFilterDropdownSubMenuFieldType - ].subFields.map((subField) => subField.subFieldName); + ].subFields + .filter((subField) => subField.isFilterable === true) + .map((subField) => subField.subFieldName); const subFieldsAreFilterable = isDefined(fieldMetadataItemUsedInDropdown) &&