alisha-1000 commented on issue #3493: URL: https://github.com/apache/streampipes/issues/3493#issuecomment-3846042091
Hi @tenthe, Thanks for reporting this. I can reproduce the behavior: when Advanced options are enabled for the Siemens S7 (Basic) adapter, the Next button remains disabled until controller-type is set, but there’s no indication that this field is required. Since the other advanced fields appear optional, this is a bit confusing from a UX perspective. Before opening a PR, I’d like to clarify the intended behavior: Is controller-type actually mandatory when Advanced options are enabled? If yes, would adding a required-field indicator or helper text be the preferred solution? If not, should the validation be relaxed so the Next button is enabled without it? Happy to take this up — once the intended behavior is confirmed, I’ll open a PR and implement the fix in line with the preferred UX and validation approach. -- 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]
