Klaudia Pałkowska

Thought so. Thanks!

Thought so. Thanks!

Is ">" intentional?

Is ">" intentional?

Oops, done.

Oops, done.

Sure, done.

Sure, done.

Yes, I've added logging.

Yes, I've added logging.

Added --no-recreate to avoid recreating db container.

Added --no-recreate to avoid recreating db container.

It doesn't work because only hapifhir and diagnostics can load spring profile from an environment variable. Updated this to set the profile in the .env file instead.

It doesn't work because only hapifhir and diagnostics can load spring profile from an environment variable. Updated this to set the profile in the .env file instead.

Changed the order of commands - it should return in one line names of containers with "build" but without "db". In the previous version, it doesn't return anything for me. Cut the end of the line b...

Changed the order of commands - it should return in one line names of containers with "build" but without "db". In the previous version, it doesn't return anything for me. Cut the end of the line because it removes from logs information which containers were removed.

Printing logs excluding those with "Resource2Db" which are inserts to a database.

Printing logs excluding those with "Resource2Db" which are inserts to a database.

OLMIS-5468: Fixed migration tests
OLMIS-5468: Fixed migration tests
this.this? Is that intentional?

this.this? Is that intentional?

Why do we map boolean to String?

Why do we map boolean to String?

OLMIS-4943: Added check if field is null before formatting the value in repquisition report
OLMIS-4943: Added check if field is null before formatting the value in repquisition report
OLMIS-4943: Fixed reportingRate report to use locale settings
OLMIS-4943: Fixed reportingRate report to use locale settings
In lines 49 and 52 we are counting the sum of quantities/soh which are never null, so I assume sum also can't be null. Previously (before my changes) it wasn't checked.

In lines 49 and 52 we are counting the sum of quantities/soh which are never null, so I assume sum also can't be null. Previously (before my changes) it wasn't checked.

Jasper studio... I'll remove it.

Jasper studio... I'll remove it.

OLMIS-4943: Fixed reports to use locale settings
OLMIS-4943: Fixed reports to use locale settings
I see, so I'm resolving my comment.

I see, so I'm resolving my comment.

Maybe use JSON.stringify instead of _.isEqual? I don't think we want to use underscore methods... Am I right, Mateusz Kwiatkowski?

Maybe use JSON.stringify instead of _.isEqual? I don't think we want to use underscore methods... Am I right, Mateusz Kwiatkowski?