[BACK-2744] Add AnyHigh and AnyLow to Summaries #127

Roukoswarf · 2023-11-08T03:42:03Z

No description provided.

ewollesen

Mostly just questions, assuming those questions don't turn up issues, then I think this is GTG.

api/mappers.go

client/types.go

patients/tide.go

client/client.go

client/types.go

ewollesen

LGTM

I'm sure you're aware, but just in case, make sure to have QA do a full regression, in case something got missed with all the pointer changes. :)

Roukoswarf added 2 commits November 8, 2023 03:41

add AnyHigh and AnyLow to summary periods and TIDE report

208585a

update travis mongo

92672a0

ewollesen previously approved these changes Nov 8, 2023

View reviewed changes

ewollesen reviewed Nov 8, 2023

View reviewed changes

fix missed patient list params

4d431f5

Roukoswarf dismissed ewollesen’s stale review via 4d431f5 November 8, 2023 20:02

Roukoswarf added 2 commits November 9, 2023 00:51

remove redundant pointers from summary handling

8b22c51

add missed summary bson tags

6776fef

ewollesen previously approved these changes Nov 10, 2023

View reviewed changes

make tags nullable again

19377c8

Roukoswarf dismissed ewollesen’s stale review via 19377c8 November 10, 2023 22:17

Roukoswarf added 3 commits December 6, 2023 16:14

Merge remote-tracking branch 'origin/master' into anyhigh-anylow

30fe8fe

update codegen

261022e

update ginkgo in makefile

0417a0a

Roukoswarf merged commit 0e82515 into master Jan 5, 2024
2 checks passed

Roukoswarf deleted the anyhigh-anylow branch January 5, 2024 04:21

Provide feedback