Skip to content

[Dynamic Form] Not handling mandatory Lookup fields well #1914

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
6gal6ler6 opened this issue Dec 5, 2024 · 3 comments
Closed

[Dynamic Form] Not handling mandatory Lookup fields well #1914

6gal6ler6 opened this issue Dec 5, 2024 · 3 comments
Assignees
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Milestone

Comments

@6gal6ler6
Copy link
Contributor

Category

[ ] Enhancement
[X] Bug
[ ] Question

Version

3.20.0

Expected / Desired Behavior / Question

When wanted to save a form that has mandatory Lookup fields, and if the fields are blank, or empty, the form should not save.

Observed Behavior

When saving a form that has mandatory Lookup fields, the form will save even those fields are empty or blank

Steps to Reproduce

Create any SPO List
Add at least one Mandatory Lookup Field
Render the dynamic form in "New" status
Fill other fields and leave blank the mandatory lookup

Related Fixes:

#1760 & #1775

Screenshots:

When saving the Item:

SavingTheForm

When saved:
image

@michaelmaillot michaelmaillot added status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. type:bug labels Feb 7, 2025
@michaelmaillot
Copy link
Collaborator

Tagging @GuidoZam for assignment

@GuidoZam
Copy link
Contributor

Hi @michaelmaillot maybe you've missed it but I've already opened a PR for this 😊

@michaelmaillot
Copy link
Collaborator

michaelmaillot commented Feb 18, 2025

Hi @GuidoZam,

Definitely saw you were on this one 😉

But I don't know why, I couldn't assign the issue to you, until you've commented it. Anyway, it's done!

@AJIXuMuK AJIXuMuK added status:fixed-next-drop Issue will be fixed in upcoming release. and removed status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. labels Mar 2, 2025
@AJIXuMuK AJIXuMuK added this to the 3.21.0 milestone Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Projects
None yet
Development

No branches or pull requests

4 participants