Skip to content

fix(ui): MemoryPrivacyPanel showToast call signature — clears 3 vue-tsc errors on base - #10984

Merged
mrveiss merged 2 commits into
Dev_new_guifrom
hotfix-memtoast
Jul 6, 2026
Merged

mrveiss merged 2 commits into
Dev_new_guifrom
hotfix-memtoast

Conversation

@mrveiss

@mrveiss mrveiss commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Thinking Path

vue-tsc-baseline was failing on every PR's merge ref with 3 TS2345 errors in MemoryPrivacyPanel.vue — pre-existing on Dev_new_gui (introduced with the memory-privacy panel, #10554). useNotificationBus().showToast is (message: string, type?: ToastType, duration?), but the panel passed { message, type } objects.

What Changed

Three showToast({ message: X, type: 'success' }) calls → showToast(X, 'success') (positional). No behavior change — the toast now actually renders the intended message/type.

Verification

vue-tsc --noEmit -p tsconfig.app.json (faithful npm ci deps + @autobot/ui resolved): the 3 MemoryPrivacyPanel.vue TS2345 errors are gone (0 remaining).

Model Used

Opus 4.8

…pe), not an object; clears 3 vue-tsc errors on Dev_new_gui
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

…uth (empty setting names + vault type tag) — clears code-quality on base (#10982 follow-up)
@mrveiss
mrveiss merged commit 6954381 into Dev_new_gui Jul 6, 2026
25 of 27 checks passed
@mrveiss
mrveiss deleted the hotfix-memtoast branch July 6, 2026 09:57
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.

1 participant