puridach-w commented on PR #26327:
URL: https://github.com/apache/superset/pull/26327#issuecomment-2106315462

   > Hey @puridach-w this is looking much better now. However, I noticed that 
you can add an empty space as a valid value for the subject. Should we probably 
strip it or allow for a minimum amount of chars? Wdyt?
   
   Hi @geido. I agree and have added the condition to trap that case. Could you 
please review it again? There's a failed job under 'Build & Publish Docker 
Images / docker-build', but I noticed there is an open PR aimed at resolving 
the issue.


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to