Skip to content

fix: Duplicate sidebar entries due to race condition#2358

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

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

Conversation

@enjeck
Copy link
Contributor

@enjeck enjeck commented Mar 4, 2026

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
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