Skip to content

fix: Duplicate sidebar entries due to race condition#2358

Merged
enjeck merged 2 commits into
mainfrom
fix/2336
Mar 4, 2026
Merged

fix: Duplicate sidebar entries due to race condition#2358
enjeck merged 2 commits into
mainfrom
fix/2336

Conversation

@enjeck

@enjeck enjeck commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

fixes #2336

When an application is created from an existing table, we concurrently triggers loadTablesFromBE() and getAllContexts()
using Promise.all. Since tables and contexts fetch data and mutate the same frontend arrays, a race condition occurred. loadTablesFromBE() and loadContextTable() both populate this.tables. Now, we explicitly check to avoid this

Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck enjeck requested review from benjaminfrueh and samin-z March 4, 2026 04:06
@enjeck enjeck requested a review from blizzz as a code owner March 4, 2026 04:06
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck enjeck merged commit 80cfa3b into main Mar 4, 2026
98 of 102 checks passed
@enjeck enjeck deleted the fix/2336 branch March 4, 2026 07:17
@enjeck

enjeck commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable1.0

@backportbot backportbot Bot added the backport-request Pending backport by the backport-bot label Apr 24, 2026
@backportbot backportbot Bot removed the backport-request Pending backport by the backport-bot label Apr 24, 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.

Duplicate table entry after "Create app from table" (same table ID shown twice)

2 participants