Skip to content

Conversation

@vvoody-
Copy link
Contributor

@vvoody- vvoody- commented Aug 14, 2023

Inline add/edit form container can be mapped to object with required properties, so creation of this value object will fail when form controls are set as required but their validation is skipped by validation scope for example when grid filter is submitted.

@f3l1x
Copy link
Member

f3l1x commented Mar 12, 2025

Can you please add some tests?

@f3l1x f3l1x added the W: Tests Waiting For Tests label Mar 12, 2025
@f3l1x f3l1x self-assigned this Mar 12, 2025
@vvoody-
Copy link
Contributor Author

vvoody- commented Mar 13, 2025

So I rewrote this fix a bit, added test and rebased branch. I'm not sure if I did it right. Should I create new pull request or something?

@f3l1x
Copy link
Member

f3l1x commented Mar 14, 2025

It's good, only one thing, can you please target this MR to master branch and not 6.x?

@f3l1x f3l1x changed the base branch from v6.x to master March 14, 2025 07:41
@f3l1x
Copy link
Member

f3l1x commented Mar 14, 2025

Ok, I've changed target branch to master and there are some conflicts.

@vvoody- vvoody- force-pushed the inline-form-mapping branch from 226bcb6 to c494ece Compare March 14, 2025 09:20
@vvoody-
Copy link
Contributor Author

vvoody- commented Mar 14, 2025

Done

@f3l1x f3l1x merged commit d423221 into contributte:master Mar 14, 2025
1 of 9 checks passed
@f3l1x
Copy link
Member

f3l1x commented Mar 14, 2025

Thank you

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

Labels

W: Tests Waiting For Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants