Skip to content

refactor(angular): remove legacy basic catalog component implementations - #2213

Draft
josemontespg wants to merge 1 commit into
angular-renderer-universal-componentsfrom
angular-clean-basic-catalog-implementations
Draft

refactor(angular): remove legacy basic catalog component implementations#2213
josemontespg wants to merge 1 commit into
angular-renderer-universal-componentsfrom
angular-clean-basic-catalog-implementations

Conversation

@josemontespg

Copy link
Copy Markdown
Collaborator

Summary

Removes legacy Angular component implementations of the Basic Catalog in favor of the universal Web Component implementations provided by @a2ui/web_core.

Changes

  • Removed legacy basic catalog Angular component files and their unit tests under renderers/angular/src/v0_9/catalog/basic/.
  • Maintained BasicCatalog and BasicCatalogBase configured with universal Web Components.
  • Updated renderers/angular/CHANGELOG.md.

Verification

  • yarn --cwd renderers/angular test:unit passed (166/166 tests).
  • yarn format:all && yarn lint:all passed cleanly.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the legacy Angular component implementations of the Basic Catalog and their associated tests in favor of universal Web Components from @a2ui/web_core. A review comment points out that the update to CHANGELOG.md under the ## Unreleased section accidentally deleted other unreleased changes (such as the toWebComponent adapter and updates to SurfaceComponent and AngularCatalog), which should be preserved.

Comment thread renderers/angular/CHANGELOG.md
@josemontespg
josemontespg marked this pull request as draft August 8, 2026 03:41
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch 2 times, most recently from d061d47 to 56a1564 Compare August 8, 2026 03:44
Comment thread renderers/angular/CHANGELOG.md
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch from 56a1564 to de97200 Compare August 8, 2026 03:46
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch from de97200 to a13dcf7 Compare August 8, 2026 04:04
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch 2 times, most recently from 45db3e9 to 244c4b8 Compare August 8, 2026 05:48
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch from 244c4b8 to b4b8d35 Compare August 8, 2026 05:50
@josemontespg
josemontespg force-pushed the angular-clean-basic-catalog-implementations branch from b4b8d35 to a183f16 Compare August 8, 2026 05:52
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