Skip to content

fix(sidebar): keep views the table listing does not carry - #2953

Merged
enjeck merged 1 commit into
mainfrom
fix/noid/missing-tables-views
Oct 14, 2026
Merged

enjeck merged 1 commit into
mainfrom
fix/noid/missing-tables-views

Conversation

@blizzz

@blizzz blizzz commented Sep 4, 2026

Copy link
Copy Markdown
Member

Views come from more source (owned, and shared) and there was a race condition that would reset the list of views instead of extending it.

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stableX.X
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@blizzz
blizzz requested a review from enjeck as a code owner September 4, 2026 20:48
@blizzz blizzz added bug Something isn't working 3. to review Waiting for reviews labels Sep 4, 2026
@blizzz

This comment was marked as low quality.

Views come from more source (owned, and shared) and there was a race
condition that would reset the list of views instead of extending it.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Assisted-by: ClaudeCode:claude-opus-5
@enjeck
enjeck force-pushed the fix/noid/missing-tables-views branch from 44b8f8d to 5cdeaa2 Compare October 12, 2026 14:46
@enjeck
enjeck merged commit 543782a into main Oct 14, 2026
66 checks passed
@enjeck
enjeck deleted the fix/noid/missing-tables-views branch October 14, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Race-condition in App.vue for loading tables and shared views

3 participants