Why would change request from UI come in separate messages (1 for duration and 1 for volume)?

How can settings be pending when you set that flag to TRUE in the code below this? And why would we want to use uf volume setting in the else part of this ternary?

else reason is Invalid Request Format

Should check if iso UF volume + UF volume < max. And if user can do multiple iso uf sessions, you would check iso UF volume + any previous iso UF volume + UF volume.
I don't know if there is a minimum iso UF volume.

Looking for "less than current" only makes sense if user can do multiple iso UF sessions in a treatment. And if that's the case, we should be looking at how much time we've already spent doing iso UF instead of how much time we've spent in the HD or HDF treatment.

I think we have to check whether the iso uf duration + treatment duration does not exceed maximum treatment duration.
And I'm not sure if there is a minimum duration for iso uf, but if there is a minimum, it probably isn't the same as the minimum treatment duration.

Just add this as an AND condition in the if statement above.

If we are in treatment end state, the treatment is completed (no need to check that) and I don't think iso UF is a modality.

Do we need this? Check with Raghu. I don't think iso UF is a modality (at least not yet).

Align all or none of the =

I don't think iso uf is considered a modality - at least not in phase 1 of development.