Skip to content

Fix dev seed page layout cache invalidation#17141

Merged
prastoin merged 2 commits intomainfrom
fix-reset-page-layout-cache-invalidation
Jan 14, 2026
Merged

Fix dev seed page layout cache invalidation#17141
prastoin merged 2 commits intomainfrom
fix-reset-page-layout-cache-invalidation

Conversation

@prastoin
Copy link
Copy Markdown
Contributor

@prastoin prastoin commented Jan 14, 2026

Introduction

followup #16962

Root cause: as now the twenty standard app installation in seeded workspace invalidates and set the page layout-xxx caches the inserted through direct repo access data in the prefill legacy page layout methods do not interact with the cache
The cache being set in prior when recompute does not result in db introspection as before as when was unset

@prastoin prastoin changed the title Fix dev seed page layout cache invaldiation Fix dev seed page layout cache invalidation Jan 14, 2026
@prastoin prastoin self-assigned this Jan 14, 2026
@prastoin prastoin marked this pull request as ready for review January 14, 2026 09:46
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 14, 2026

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:32474

This environment will automatically shut down when the PR is closed or after 5 hours.

Copy link
Copy Markdown
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

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

LGTM

@prastoin prastoin added this pull request to the merge queue Jan 14, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 14, 2026
@prastoin prastoin added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 7070426 Jan 14, 2026
62 of 63 checks passed
@prastoin prastoin deleted the fix-reset-page-layout-cache-invalidation branch January 14, 2026 12:20
@twenty-eng-sync
Copy link
Copy Markdown

Hey @prastoin! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

1 similar comment
@twenty-eng-sync
Copy link
Copy Markdown

Hey @prastoin! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

lucasbordeau pushed a commit to vasu1303/twenty that referenced this pull request Jan 21, 2026
# Introduction

followup twentyhq#16962

Root cause: as now the twenty standard app installation in seeded
workspace invalidates and set the page layout-xxx caches the inserted
through direct repo access data in the prefill legacy page layout
methods do not interact with the cache
The cache being set in prior when recompute does not result in db
introspection as before as when was unset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants