fix: accept deep partials in patchValue and reset methods

[rafaelss95]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Currently if we try to patchValue a nested form, we got a type error if missing any control in nested structure.

Sample:

this.formGroup = this.formBuilder.group({
  a: this.formBuilder.group({
    b: [1, [Validators.required]],
    c: '2'
  })
});

this.formGroup.patchValue({
  a: { c: '9' } // Error, missing "b"
});

PS: The same applies for formGroup#reset (the reset of FormArray was not using Partial, so I didn't touch it).

Issue Number: N/A

What is the new behavior?

No errors while patchValue for nested structures.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

[rafaelss95]

Hello @NetanelBasal sry to bump you, but is there any chance of having this reviewed?

[NetanelBasal]

@itayod, please have a look

[itayod]

@rafaelss95 LGTM, well done!

[itayod]

please resolve the conflicts and then I think we can merge it.

[rafaelss95]

Done, @itayod :)

[itayod]

Thanks for the quick response :)

[rafaelss95]

@itayod could you publish a version with this? I'm about to use it now 🚀