AnzhiZhang commented on PR #4006:
URL: https://github.com/apache/texera/pull/4006#issuecomment-3514958769

   > Generally, I don't understand the update, I checked out your branch and 
compared it with `main` and even in `main` if the required field is empty, we 
still get error. Its because of `required=true` in @JsonProperty. Frontend 
handle the required ones so even you made these changes frontend still show old 
message.
   
   `required=true` does not add necessary restrictions to the JSON schema and 
should be handled in the frontend. When the field is an empty string, the 
frontend still shows to the user that the workflow is valid. Here is a video to 
demonstrate this behavior and details behind it: 
https://github.com/apache/texera/issues/3692#issue-3350539271.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to