Skip to content
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

[Dynamic Form] Always Show Required Field Validation Error In "FormDisplayMode.Edit" Mode #1775

Open
wuxiaojun514 opened this issue Feb 26, 2024 · 3 comments
Assignees
Labels
status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. type:bug

Comments

@wuxiaojun514
Copy link
Contributor

Category

[ ] Enhancement

[x] Bug

[ ] Question

Version

Please specify what version of the library you are using: [ 3.17 ]. It works well in [3.16]
Dynamic Form refactored a lot in 3.17

Expected / Desired Behavior / Question

Ideally when you edit an existing item with "Dynamic Form", it should not show "You can't leave this blank." error if this field already has data.

Observed Behavior

When in Edit mode (PageType=6), Dynamic Form control will always show "You can't leave this blank." error on required field even it already has existing data on current item.
image

Steps to Reproduce

  1. Choose a list which has a required field (e.g. title) and create a new item. Make sure this new created item has data on required field.
  2. Put "Dynamic Form" in the spfx form customization, no special settings.
     <DynamicForm
          context={this.props.context}
          listId={this.props.context.list.guid.toString()}
          listItemId={this.props.context.itemId}
          onSubmitted={(listItemData: any) => {
            console.log(listItemData);       
              setTimeout(() => {
                this.props.onSave();
              }, 1000);
          }} 
          onListItemLoaded={async (listItemData: any) => { // eslint-disable-line @typescript-eslint/no-explicit-any
            console.log(listItemData);
          }} />
  1. Update EditForm Config in "serve.json" to point to this new created item, then launch it with edit form config
    image

  2. You will see "You can't leave this blank." error on this item even it already has data on this field
    image

@ghost
Copy link

ghost commented Feb 26, 2024

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label Feb 26, 2024
wuxiaojun514 pushed a commit to wuxiaojun514/sp-dev-fx-controls-react that referenced this issue Feb 27, 2024
wuxiaojun514 pushed a commit to wuxiaojun514/sp-dev-fx-controls-react that referenced this issue Feb 29, 2024
@michaelmaillot michaelmaillot added status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. type:bug and removed Needs: Triage 🔍 labels Apr 25, 2024
@NishkalankBezawada
Copy link
Contributor

Hello,

We are also facing the same error with version 3.18.1. @wuxiaojun514 Any update on the fix?

Thanks,
Nishkalank

@wuxiaojun514
Copy link
Contributor Author

Hi @NishkalankBezawada I created a PR 1781 for it. But it seems waiting some other PR 1726 getting approval first. Right now my fix has not been merged into Dev branch yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. type:bug
Projects
None yet
Development

No branches or pull requests

3 participants