Skip to content

refactor(core): reuse setupStaticAttributes in ComponentRef#59572

Closed
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_component_ref_attrs
Closed

refactor(core): reuse setupStaticAttributes in ComponentRef#59572
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_component_ref_attrs

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

This change refactor how the dynamically created component deals with attributes in order to reuse the existing setupStaticAttributes logic (instead of having specific and similar code).

@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Jan 16, 2025
@ngbot ngbot bot added this to the Backlog milestone Jan 16, 2025
@pkozlowski-opensource pkozlowski-opensource added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jan 16, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_component_ref_attrs branch from 73d5bf1 to a23d9e5 Compare January 16, 2025 17:19
@pkozlowski-opensource pkozlowski-opensource added action: merge The PR is ready for merge by the caretaker action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer action: merge The PR is ready for merge by the caretaker labels Jan 16, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_component_ref_attrs branch 2 times, most recently from e3c2034 to 256ac6d Compare January 17, 2025 13:52
@pkozlowski-opensource pkozlowski-opensource added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jan 17, 2025
This change refactor how the dynamically created component
deals with attributes in order to reuse the existing
setupStaticAttributes logic (instead of having specific
and similar code).
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_component_ref_attrs branch from 256ac6d to f0b6ea1 Compare January 17, 2025 14:33
@pkozlowski-opensource pkozlowski-opensource added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Jan 17, 2025
@pkozlowski-opensource
Copy link
Copy Markdown
Member Author

caretaker note: I've got a "green" TGP for this one.

@AndrewKushnir
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit 35e5bbb.

The changes were merged into the following branches: main, 19.1.x

AndrewKushnir pushed a commit that referenced this pull request Jan 17, 2025
This change refactor how the dynamically created component
deals with attributes in order to reuse the existing
setupStaticAttributes logic (instead of having specific
and similar code).

PR Close #59572
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
…59572)

This change refactor how the dynamically created component
deals with attributes in order to reuse the existing
setupStaticAttributes logic (instead of having specific
and similar code).

PR Close angular#59572
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants