•  

Comment Results

Review Name Created Custom Fields Content
HD-DEN-7395-1 19 Apr 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-13460-2 22 Sep 2022

RESOLVED in CODE WALKTHROUGH.

DG-DEN-5963-1 19 Apr 2021

RESOLVED in CODE WALKTHROUGH.

HD-DEN-8679-1 29 Jul 2021

RESOLVED in CODE WALKTRHOUGH.

DIALIN-DEN-7395-1 19 Apr 2021

Addressed.

DIALIN-DEN-7395-1 19 Apr 2021

Addressed.

HD-DEN-8679-1 29 Jul 2021

RESOLVED in CODE WALKTRHOUGH.

SRVC-DEN-7335-1 21 Apr 2021

Remove extra line.

DG-DEN-11750-1 02 Mar 2022

This will be addressed once we were working on deciding the number of rinses.

DG-DEN-7568-1 23 Apr 2021

Done.

DG-DEN-9054-1 29 Jul 2021

Bamboo will do the copyrights. It is not turned on right now but it will automatically.

DG-DEN-11928-1 01 Mar 2022

Renamed roPumpSpeed to roPumpFlushBubblesSpeed and updated comments

DG-DEN-9054-1 30 Jul 2021

RESOLVED in CODE WALKTHROUGH.

UI-DEN-9411-1 12 Aug 2021

Updated.

HD-DEN-14344-5 02 Dec 2022

Fixed. Thanks!

UI-DEN-9411-1 12 Aug 2021

Updated.

RO-LDT-1242-1 03 Sep 2025

fixed thanks!

UI-DEN-9411-1 12 Aug 2021

Comment doesn't match code.

DG-DEN-7802-1 12 Aug 2021

Done. Should we remove this build flag all together since no V2 system available anymore?

UI-DEN-9411-1 12 Aug 2021

Missing brief.

DG-DEN-11928-1 01 Mar 2022

Added param: testValue, baseValue, percentFactor

HD-DEN-11114-1 04 Jan 2022

Done.

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11098-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11114-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11114-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11114-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11114-1 04 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11750-2 02 Mar 2022

Should this be in an else below?

DG-DEN-11114-1 04 Jan 2022

Done.

HD-DEN-11750-2 02 Mar 2022

It doesn't appear to build for me.

DIALIN-DEN-11114-1 05 Jan 2022

Done.

DG-DEN-11114-1 05 Jan 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11750-2 02 Mar 2022

Can leave uninitialized here. It is initialized in init function below.

DG-DEN-11114-1 05 Jan 2022

RESOLVED in CODE WALKTHROUGH.

DG-DEN-11928-1 01 Mar 2022

Sent Dara email asking him to put those numbers in the DG calibration record.

UI-BUG-S62-1 02 Feb 2022

RESOLVED.

DG-DEN-11750-1 22 Feb 2022

Could we move these fields outside of the for loop? They dont need to be initialized NUM_OF_FANS_NAMES times.

HD-DEN-11250-1 01 Feb 2022

RESOLVED in CODE WALKTHROUGH.

HD-DEN-11250-1 31 Jan 2022

Bolus rate is fixed now so no range check is needed. Keep check for home though.

DG-DEN-11750-1 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11250-1 15 Feb 2022

RESOLVED in CODE WALKTHROUGH.

UI-BUG-S62-1 01 Feb 2022

The Arterial and Venous sliders are RangeSlider and are handled differently.
The extra code is removed.

DIALIN-BUG-S63-1 26 Jan 2022

Mode 4 for the HD is Treatment parameters. Every time this is called the UI will flicker to the Patient ID screen then back to Home. Is that what we want?

HD-DEN-11750-2 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

HD-DEN-11750-2 03 Mar 2022

RESOVLED IN CODE WALKTHROUGH.

UI-BUG-S63-2 14 Feb 2022

Thanks,
Aligned.

DG-DEN-11750-1 02 Mar 2022

Should this happen in the previous if scope to simplify?

UI-BUG-S63-2 14 Feb 2022

Thanks,
Updated.

HD-DEN-11750-2 02 Mar 2022

Have you tried a release build to see if we have this right?