Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
It is a little bit concerning: Could you please test/describe the following scenarios: *What if we have more than one Heparin type supported? *What if none (incorrectly) has been defined in the c...

It is a little bit concerning:
Could you please test/describe the following scenarios:

  • What if we have more than one Heparin type supported?
  • What if none (incorrectly) has been defined in the configuration (do we get an index out of bounds)?
We will address this later on code modularizations. RESOLVED.

We will address this later on code modularizations.
RESOLVED.

LDT-3003 End of Treatment - SW - 01 - R&I - 03: SDD - Design Diagram - plantuml

update message id

LEAHI-APPLICATION-LDT-3000_End of Treatment - SW - 01 - R&I
LEAHI-APPLICATION-LDT-3000_End of Treatment - SW - 01 - R&I
LDT-3003 End of Treatment - SW - 01 - R&I - 03: SDD - Design Diagram - plantuml

add SDD diagram

Insittutional

add message IDs

  1. … 5 more files in changeset.
LDT-2962 Heparin - SW - 01 - R&I - 05: DEV - Feature Implementation

add message IDs

  1. … 3 more files in changeset.
I believe any related view could have provided a timer since they are inherited from QObject. That is fine for now but consider using our C++ view codes for such a thing.

I believe any related view could have provided a timer since they are inherited from QObject.
That is fine for now but consider using our C++ view codes for such a thing.

Thank you .

Thank you .

pop(null) will unwind the stack and go back to main treatment. I tested and added a comment in code as well

pop(null) will unwind the stack and go back to main treatment. I tested and added a comment in code as well

updated to statusColor

updated to statusColor

added

added

used for the 1 min user activity timeout to move back to main treatment from heparin.. Its a requirement LEAHI-PRS-666

used for the 1 min user activity timeout to move back to main treatment from heparin.. Its a requirement LEAHI-PRS-666

updated

updated

updated

updated

yes good catch. updated

yes good catch. updated

moved

moved

removed

removed

done

done

Please align.

Please align.

Please use the mainTreatment screen as the default. If necessary, could you create a specific enum to clean up the stack (which I do not believe is needed.)

Please use the mainTreatment screen as the default.
If necessary, could you create a specific enum to clean up the stack (which I do not believe is needed.)

You no longer need this after you move the heparin screen from the sections folder.

You no longer need this after you move the heparin screen from the sections folder.

Shouldn't ""Unfractionated 1000"" come from the configurations?

Shouldn't ""Unfractionated 1000"" come from the configurations?

We should add the qsTr()

We should add the qsTr()

Move this outside of the sections folder, since it is no longer a section and has its own dedicated screen.

Move this outside of the sections folder, since it is no longer a section and has its own dedicated screen.

We should use "min" for minutes and not "mins". Same for "hrs" => "hr".

We should use "min" for minutes and not "mins".
Same for "hrs" => "hr".

What is this file used for? And please add a brief description for the file comment.

What is this file used for?
And please add a brief description for the file comment.

Please put comments for your division number.

Please put comments for your division number.

statusColor is so general, but is bound to the heparinStatusColor. Please either change the status color to be heparin only or change the heparinStatus Color name to be more general. The latter is ...

statusColor is so general, but is bound to the heparinStatusColor.
Please either change the status color to be heparin only or change the heparinStatus Color name to be more general.
The latter is better.

okay will do

okay will do