Skip to content

Answer:61#1449

Open
tsironis13 wants to merge 3 commits intotomalaforge:mainfrom
tsironis13:forms-simplest-signal-form-challenge
Open

Answer:61#1449
tsironis13 wants to merge 3 commits intotomalaforge:mainfrom
tsironis13:forms-simplest-signal-form-challenge

Conversation

@tsironis13
Copy link
Contributor

No description provided.

@github-actions github-actions bot added 61 simple signal form answer answer contributor highly contributing to the project to be reviewed PR requests a review labels Mar 9, 2026
private readonly _initialData: UserData = {
name: '',
lastname: '',
age: NaN,
Copy link
Owner

Choose a reason for hiding this comment

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

i need to update the project, so null will be allowed 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok. In addition, if y update the angular to 21.2, the form submittion logic can be implemented in a declarative way using the formRoot directive.

@tomalaforge tomalaforge removed the to be reviewed PR requests a review label Mar 9, 2026
@tomalaforge
Copy link
Owner

@tsironis13 if you rebase your branch and run pnpm i --frozen-lockfile, you will be on angular 21.2.2 so you'll be able to use the latest improvement 🚀

@tsironis13
Copy link
Contributor Author

tsironis13 commented Mar 10, 2026

@tsironis13 if you rebase your branch and run pnpm i --frozen-lockfile, you will be on angular 21.2.2 so you'll be able to use the latest improvement 🚀

@tomalaforge thanks Thomas! code updated.

action: async () => {
if (this.userForm().valid()) {
this.setSubmittedData();
}
Copy link
Owner

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

61 simple signal form answer answer contributor highly contributing to the project to be reviewed PR requests a review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants