Label field need validation on the FieldStorageAddForm.

Issue created by @utkarsh_33
Merge request !5492Added a check for label field → (Closed) created by utkarsh_33
Comment over 1 year ago →
utkarsh_33
Status changed to Needs review over 1 year ago11:54am 21 November 2023
Comment over 1 year ago →
utkarsh_33
Status changed to Needs work over 1 year ago12:55pm 21 November 2023
Comment over 1 year ago →
🇮🇳India narendraR Jaipur, India
Looks good. Added `needs tests` tag to add a test to validate this.
Status changed to Needs review over 1 year ago4:48am 22 November 2023
Comment over 1 year ago →
utkarsh_33
Comment over 1 year ago →
utkarsh_33
Added test coverage.
Status changed to RTBC over 1 year ago6:20am 22 November 2023
Comment over 1 year ago →
🇮🇳India narendraR Jaipur, India
Status changed to Needs work over 1 year ago8:42am 30 November 2023
Comment over 1 year ago →
🇫🇮Finland lauriii Finland
Posted feedback in the MR.
Comment over 1 year ago →
🇮🇳India omkar.podey
omkar.podey → made their first commit to this issue’s fork.
Status changed to Needs review over 1 year ago12:42pm 30 November 2023
Comment over 1 year ago →
🇮🇳India omkar.podey
Status changed to RTBC over 1 year ago1:03pm 30 November 2023
Comment over 1 year ago →
🇮🇳India Nitin shrivastava
Hi @omkar.podey, MR #10 has been successfully applied, resolving the issue. However, if Someone try to save without filling in the field data, an error message will appear.

lauriii → committed 5ed165d5 on 11.x

Issue #3403101 by Utkarsh_33, omkar.podey, Nitin shrivastava, narendraR...

lauriii → committed b52f3a06 on 10.2.x

Issue #3403101 by Utkarsh_33, omkar.podey, Nitin shrivastava, narendraR...

Status changed to Fixed over 1 year ago7:48am 1 December 2023
Comment over 1 year ago →
🇫🇮Finland lauriii Finland
Committed 5ed165d and pushed to 11.x. Also cherry-picked to 10.2.x. Thanks!
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Merge Requests