Skip to content

Conversation

@fahrigedik
Copy link
Member

Description

Resolves #23300 (write the related issue number if available)

TODO: As part of the migration to standalone component structure, this PR replaces internal module imports with their corresponding standalone components, directives, and pipes.

Checklist

  • I fully tested it as developer / designer and created unit / integration tests
  • I documented it (or no need to document or I will create a separate documentation issue)

How to test it?

Build the full workspace
Run the following command to ensure the entire workspace builds correctly with updated standalone imports:
`yarn build:all
There should be no compilation or type errors.

Start the application
Launch the application using:
`yarn start
Application should load without issues in the browser.

… package

Replaced CommonModule and RouterModule imports with more granular standalone Angular feature imports (NgClass, NgStyle, NgIf, NgTemplateOutlet, RouterLink, etc.) across multiple shared components. This change improves tree-shaking and aligns with Angular's standalone component best practices.
Added AsyncPipe to the imports array of several Angular components to support async data binding in templates. This change improves template handling of observables and aligns with Angular best practices.
Copy link
Contributor

@sumeyyeKurtulus sumeyyeKurtulus left a comment

Choose a reason for hiding this comment

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

Everything seems right other than some small points I caught. You could also check for LocalizationModule imports that need to be replaced with relative localization pipes 🙌🏻

That would be better if you start modifying staff after solving the conflicts.

@yagmurcelk yagmurcelk merged commit 2a273ff into dev Jul 25, 2025
2 of 3 checks passed
@yagmurcelk yagmurcelk deleted the issue-23300 branch July 25, 2025 12:52
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.

Angular - Replace redundant module usage for standalone components

4 participants