Skip to content

Alias Utils::SmartBlob to OSS::Smart::Blob component (for bc) - #469

Merged
phndiaye merged 3 commits into
masterfrom
pn/alias-smart-blob-from-oss-components
Jul 20, 2026
Merged

Alias Utils::SmartBlob to OSS::Smart::Blob component (for bc)#469
phndiaye merged 3 commits into
masterfrom
pn/alias-smart-blob-from-oss-components

Conversation

@phndiaye

@phndiaye phndiaye commented Jun 9, 2026

Copy link
Copy Markdown
Member

What does this PR do?

With the smart-blob component being moved to oss-components, this PR ensures Utils::SmartBlob is just an alias of OSS::Smart::Blob for backward compatibility purposes.

What are the observable changes?

This block from the local dummy app still displays correctly:

<Utils::SmartBlob @loading={{this.blobLoading}} @size="sm" />
<Utils::SmartBlob @loading={{this.blobLoading}} />
<Utils::SmartBlob @loading={{this.blobLoading}} @size="xl" />

Screenshot 2026-06-09 at 17 40 06

🧑‍💻 Developer Heads Up

⚡ Since we are using Ember Octane now:

  • Feel free to migrate existing components to Glimmer Components.
  • Write new ones exclusively in it.

Useful Resource : Ember Octane vs Classic Cheat Sheet

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

@phndiaye phndiaye self-assigned this Jun 9, 2026
@phndiaye
phndiaye requested a review from a team as a code owner June 9, 2026 15:40
@phndiaye
phndiaye requested review from OwenCoogan, edouardmisset and olxmpe and removed request for a team June 9, 2026 15:40
@phndiaye
phndiaye force-pushed the pn/alias-smart-blob-from-oss-components branch from 3bc6b5f to 3a22737 Compare July 20, 2026 13:55
@phndiaye
phndiaye merged commit c86beeb into master Jul 20, 2026
5 checks passed
@phndiaye
phndiaye deleted the pn/alias-smart-blob-from-oss-components branch July 20, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants