From 3c5d139a1a90e4dc7df81e0bb4d67a4719ee9923 Mon Sep 17 00:00:00 2001 From: Mihai <163020214+umihai29@users.noreply.github.com> Date: Fri, 28 Feb 2025 10:30:12 +0200 Subject: [PATCH] fix payload when you create a new view from a shared one (#98) --- .../FiltersContainer/FiltersContainer.tsx | 35 ++++++++++--------- .../ContactDetails/ContactDetails.tsx | 2 +- src/store/TableViewDefs/TableViewDef.store.ts | 7 +++- 3 files changed, 26 insertions(+), 18 deletions(-) diff --git a/src/routes/finder/src/components/FiltersContainer/FiltersContainer.tsx b/src/routes/finder/src/components/FiltersContainer/FiltersContainer.tsx index 8cf16dea..8c03d770 100644 --- a/src/routes/finder/src/components/FiltersContainer/FiltersContainer.tsx +++ b/src/routes/finder/src/components/FiltersContainer/FiltersContainer.tsx @@ -27,7 +27,6 @@ export const FiltersContainer = observer(() => { const tableViewType = tableViewDef?.value.tableType; const tableType = tableViewDef?.value?.tableType; - const isPreset = tableViewDef?.value?.isPreset; const filters = tableViewDef?.getFilters()?.AND?.length > 0; const handleAddToMyViews: MouseEventHandler = (e) => { @@ -77,36 +76,40 @@ export const FiltersContainer = observer(() => { store.ui.isEditingDefaultFilters && 'bg-grayModern-50', )} > + {tableViewDef?.hasFilters() && !store.ui.isEditingDefaultFilters && ( + <> + + {filters && ( + + )} + + + )} {tableViewDef?.hasDefaultFilters() && !store.ui.isEditingDefaultFilters && ( <> - )} {!store.ui.isEditingDefaultFilters && ( <> - {tableViewDef?.hasFilters() && ( - - )} - {isPreset && filters && ( - - )} {filters && } {tableViewType && ( diff --git a/src/routes/src/components/ContactDetails/ContactDetails.tsx b/src/routes/src/components/ContactDetails/ContactDetails.tsx index 022900f5..43d2bba1 100644 --- a/src/routes/src/components/ContactDetails/ContactDetails.tsx +++ b/src/routes/src/components/ContactDetails/ContactDetails.tsx @@ -197,7 +197,7 @@ export const ContactDetails = observer( dataTest='org-people-contact-name' onFocus={(e) => e.target.select()} onKeyDown={(e) => e.stopPropagation()} - className='placeholder:font-medium font-medium min-w-[60px] w-[200px]' + className='placeholder:font-medium font-medium min-w-[60px] w-[300px]' onBlur={() => { contactDetails.changeContactName(); }} diff --git a/src/store/TableViewDefs/TableViewDef.store.ts b/src/store/TableViewDefs/TableViewDef.store.ts index 0fd50620..ec596f39 100644 --- a/src/store/TableViewDefs/TableViewDef.store.ts +++ b/src/store/TableViewDefs/TableViewDef.store.ts @@ -227,6 +227,9 @@ export class TableViewDefStore extends Store { ) => { const favoritePreset = this.getById(id)?.getPayloadToCopy(); + const favoriteDefaultFilters = this.getById(id)?.getDefaultFilters().AND; + const favoriteFilters = this.getById(id)?.getFilters().AND; + const newTableViewDef = new TableViewDef( this, TableViewDef.default({ @@ -241,7 +244,9 @@ export class TableViewDefStore extends Store { isPreset: false, isShared, filters: '', - defaultFilters: favoritePreset?.filters || '', + defaultFilters: JSON.stringify({ + AND: [...favoriteDefaultFilters, ...favoriteFilters], + }), }), );