leahi-application / sources / gui / qml / pages

pretreatment

Clone Tools

Stats

Commits this week: 1
Total Committers: 5
Last Commit: 31 Dec 69
Commits this week: 1

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 16 Dec
4:12 pm

Behrouz NematiPour

RESOLVED.

RESOLVED.

4:07 pm

Nicholas Ramirez

fixed

fixed

Friday 12 Dec
1:43 am

Behrouz NematiPour

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)?
1:39 am

Behrouz NematiPour

We will address this later on code modularizations. RESOLVED.

We will address this later on code modularizations.
RESOLVED.

Wednesday 10 Dec
4:54 pm

Behrouz NematiPour

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.

4:50 pm

Behrouz NematiPour

Thank you .

Thank you .

4:46 pm

Nicholas Ramirez

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

4:31 pm

Nicholas Ramirez

updated to statusColor

updated to statusColor

4:30 pm

Nicholas Ramirez

added

added

4:30 pm

Nicholas Ramirez

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

4:29 pm

Nicholas Ramirez

updated

updated

4:29 pm

Nicholas Ramirez

updated

updated

4:27 pm

Nicholas Ramirez

yes good catch. updated

yes good catch. updated

4:27 pm

Nicholas Ramirez

moved

moved

4:26 pm

Nicholas Ramirez

removed

removed

4:26 pm

Nicholas Ramirez

done

done

3:59 pm

Behrouz NematiPour

Please align.

Please align.

3:56 pm

Behrouz NematiPour

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.)

3:53 pm

Behrouz NematiPour

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.

3:51 pm

Behrouz NematiPour

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

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

3:49 pm

Behrouz NematiPour

We should add the qsTr()

We should add the qsTr()

3:48 pm

Behrouz NematiPour

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.

3:37 pm

Behrouz NematiPour

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".

3:28 pm

Behrouz NematiPour

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.

3:26 pm

Behrouz NematiPour

Please put comments for your division number.

Please put comments for your division number.

3:26 pm

Behrouz NematiPour

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.

3:07 pm

Nicholas Ramirez

okay will do

okay will do

3:07 pm

Nicholas Ramirez

okay acknowledged

okay acknowledged

1:44 pm

Behrouz NematiPour

No Action For Now: Same for the vError param: https://devapps.diality.us/cru/#LEAHI-APPLICATION-LDT-2727-1CFR-76762

No Action For Now:
Same for the vError param:
https://devapps.diality.us/cru/#LEAHI-APPLICATION-LDT-2727-1CFR-76762

1:43 pm

Behrouz NematiPour

No Action For Now: Please always use mError, since it is local to the script function, to be consistent with the rest. Also, usually it is a number, not a bool. It is fine for now, but later we sh...

No Action For Now:
Please always use mError, since it is local to the script function, to be consistent with the rest.
Also, usually it is a number, not a bool.

It is fine for now, but later we should create a decorator to cover these types of similarities.