✨ Frontend: Expose ``inputs required`` property

odeimaiz commented 1 month ago

What do these changes do?

This PR exposes the node's inputsRequired attribute to the frontend. Users will now be able to make inputs required.

RequiredInputs

How to test

Dev-ops checklist

[ ] No ENV changes or I properly updated ENV (read the instruction)

odeimaiz commented 1 month ago

Maybe you can add a tooltip saying what happens when you set required.

The tooltip will say: "Required input: Without it, the service will not start/run."

@elisabettai Is that fine?

elisabettai commented 1 month ago

Maybe you can add a tooltip saying what happens when you set required.

The tooltip will say: "Required input: Without it, the service will not start/run."

@elisabettai Is that fine?

It sounds good. I also just noticed that you change the string when the option is selected, and it is a bit confusing (like it was for Service Autostart). I know you don't like a switch button, and without adding a color to it, it might be more confusing. What about:

"Mandatory input: 🟩 " (it is not required)
"Mandatory input: ✅ " (it is required)

If you go in that direction, than the tooltip should say: "Mandatory input: without it, the connected service will not start/run"