Skip to content

refactor(docs-infra): migrate adev to signal-based inputs - #58453

Closed
pkozlowski-opensource wants to merge 1 commit into
angular:mainfrom
pkozlowski-opensource:adev_input_migration
Closed

pkozlowski-opensource wants to merge 1 commit into
angular:mainfrom
pkozlowski-opensource:adev_input_migration

Conversation

@pkozlowski-opensource

Copy link
Copy Markdown
Member

This change is the result of running the automated input migration on adev (with the safe flags).

This change is the result of running the automated input
migration on adev (with the safe flags).
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Oct 31, 2024
@ngbot ngbot Bot added this to the Backlog milestone Oct 31, 2024
@sheikalthaf

Copy link
Copy Markdown
Contributor

@pkozlowski-opensource FYI it is not updated the relevant html files.

@pkozlowski-opensource

Copy link
Copy Markdown
Member Author

@sheikalthaf yep. Opened #58455 to track this

@sheikalthaf

Copy link
Copy Markdown
Contributor

@Input() item?: SearchResult;
@Input() disabled = false;
readonly item = input<SearchResult>();
readonly disabled = input(false);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is actually breaking because disabled is overriding the definition of Highlightable

@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 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: docs-infra Angular.dev application and infrastructure state: blocked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants