Skip to content

fix(hub): drop per-concept success toasts on Hub import - #89

Merged
tomasz-zajac-oss merged 1 commit into
mainfrom
fix/hub-import-no-success-toasts
Sep 25, 2026
Merged

tomasz-zajac-oss merged 1 commit into
mainfrom
fix/hub-import-no-success-toasts

Conversation

@tomasz-zajac-oss

Copy link
Copy Markdown
Collaborator

Summary

Importing a blueprint also imports its related concepts. Each of those imports pushed an info toast Imported "<name>". The E-commerce blueprint has 18 related concepts, so the toasts piled up over the template-parameter form that opens right after.

These toasts told the user nothing new:

  • the import modal already marks each concept as imported ("Imported" / "All N concepts imported!");
  • Radical Forge's Hub cards already switch to "✓ Imported".

This PR removes the success toast from both import paths (HubImportModal and importConcept). Failure toasts, such as "Failed to load …", are unchanged.

Type of change

  • Bug fix

Checklist

  • npm test passes (408/408)
  • npm run typecheck passes with no errors
  • New or changed behaviour is covered by tests. Not covered: this only removes UI toasts. It was checked manually (see below).

Test plan

  • In npm run dev:web with headless Chromium, imported the E-commerce blueprint and its related concepts via ?hub=bp-ecommerce-platform:
    • 0 toasts while the template-parameter form is open (before this change, a stack of "Imported …" toasts covered it)
    • 0 toasts after confirming the import
    • the modal still shows "All 1 concept imported!", and the model has all nodes
    • no console errors

🤖 Generated with Claude Code

Importing a blueprint also imports its related concepts, and each import
pushed an "Imported "<name>"" info toast. With a blueprint's dozen-plus
related concepts, the stack of toasts covered the template-parameter form
that opens right after.

The success toast added nothing: the import modal already marks each
concept as imported ("Imported" / "All N concepts imported!"), and Radical
Forge's Hub cards switch to "Imported". Removed it from both import paths
(HubImportModal and importConcept); failure toasts stay.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tomasz-zajac-oss
tomasz-zajac-oss merged commit 92d49dc into main Sep 25, 2026
2 checks passed
@tomasz-zajac-oss
tomasz-zajac-oss deleted the fix/hub-import-no-success-toasts branch September 25, 2026 10:13
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