Skip to content

fix: Fix content not displaying by adding an immediate to sync watcher in the app#1277

Open
odalamkundika-hue wants to merge 6 commits intomainfrom
1264-cms-editcontent-issues
Open

fix: Fix content not displaying by adding an immediate to sync watcher in the app#1277
odalamkundika-hue wants to merge 6 commits intomainfrom
1264-cms-editcontent-issues

Conversation

@odalamkundika-hue
Copy link
Collaborator

No description provided.

@odalamkundika-hue odalamkundika-hue linked an issue Jan 7, 2026 that may be closed by this pull request
2 tasks
@odalamkundika-hue odalamkundika-hue force-pushed the 1264-cms-editcontent-issues branch from 7dee63e to e7a37e5 Compare January 20, 2026 07:40
:id="language._id"
:key="language._id"
class="flex w-full items-center p-3"
transition="duration-200 hover:bg-yellow-100/50 dark:hover:bg-yellow-100/25"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this change

:id="language._id"
:key="language._id"
class="flex w-full cursor-pointer items-center gap-1 p-3"
transition="duration-200 hover:bg-yellow-100/50 dark:hover:bg-yellow-100/25"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this change

:key="language._id"
@click="quickLanguageSwitch(language._id)"
class="flex cursor-pointer select-none items-center gap-2 px-4 py-2 text-sm leading-6 text-zinc-800 hover:bg-zinc-50 dark:text-white dark:hover:bg-slate-600"
class="flex cursor-pointer select-none items-center gap-2 px-4 py-2 text-sm leading-6 text-zinc-800 hover:bg-yellow-100/50 dark:text-white dark:hover:bg-yellow-100/25"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this change

'mb-0 cursor-default bg-zinc-200 shadow': isActive && !isCardCollapsed,
'border-1.5 cursor-default bg-white shadow': isActive && isCardCollapsed,
'border bg-white/80 hover:bg-white/100': !isActive,
'hover:border-white-500 border bg-white/80 transition duration-200 hover:bg-zinc-100':
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try to avoid transitions where possible to keep the app and CMS as ligthweight as possible

@MrDirkelz MrDirkelz force-pushed the 1264-cms-editcontent-issues branch from b1e965d to 07d2228 Compare February 3, 2026 14:03
@MrDirkelz
Copy link
Collaborator

Meeting with @odalamkundika-hue on Feb 4 Wednesday 2026 (notes for discussion)

When saving/creating documents, the processContentDoc function adds the parent doc's tags to the content document it is adding with no checks - it just does it

When duplicating, we want to remove the cached parent tags on the content documents

We tried doing thatt in the client code. But it is overriden from the processContentDoc function.

@ivanslabbert do you have a way we can approach this?

@MrDirkelz
Copy link
Collaborator

Meeting with @odalamkundika-hue on Feb 4 Wednesday 2026 (notes for discussion)

When saving/creating documents, the processContentDoc function adds the parent doc's tags to the content document it is adding with no checks - it just does it

When duplicating, we want to remove the cached parent tags on the content documents

We tried doing thatt in the client code. But it is overriden from the processContentDoc function.

@ivanslabbert do you have a way we can approach this?

Single source of truth for cached docs in api
Api handles caching of fields in dto

@MrDirkelz MrDirkelz assigned MrDirkelz and unassigned MrDirkelz Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CMS: EditContent issues

3 participants