Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

TIDE Pagination #117

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

TIDE Pagination #117

wants to merge 31 commits into from

Conversation

Roukoswarf
Copy link
Contributor

No description provided.

@Roukoswarf
update travis and golang version
1ab8f58
@Roukoswarf
add offset periods to client and server
91eaa44
@Roukoswarf
fix circular import while keeping simpler casting
0d5fa4a
@Roukoswarf
cleanup some pointer casting
eb75322
@Roukoswarf
inital TIDE api rigging
898acf7
@Roukoswarf
TIDE API draft
390932a
@Roukoswarf
revert some build tweaks
7b5357f
@Roukoswarf
more build changes
5cf874f
@Roukoswarf
add tide_report to policy
b048dfc
@Roukoswarf
pass pointer to decode
49e2f9c
@Roukoswarf
update deps, remove key requirements from summary update api
3865b6f
@Roukoswarf
add meetingTargets
d4ba27a
@Roukoswarf
update vendor
23a59ac
@Roukoswarf
replace Decode with All on tide query
1f4d9a7
@Roukoswarf
fix length of category slice
808fe51
@Roukoswarf
add some query debug for sort/match on patient list and TIDE
320243f
@Roukoswarf
add bson tags to summaries again
b8fd5cb
@Roukoswarf
add debug to TIDE report
3124cad
@Roukoswarf
handle clinic and tag objectId query correctly
09e2d83
@Roukoswarf
clean up date range query for TIDE
ceaec4b
@Roukoswarf
add query logging
449f924
@Roukoswarf
fix category query types
dbed41a
@Roukoswarf
change dropInTimeInTargetPercent condition to <-0.15
f22732e
@Roukoswarf
make tags $all instead of $in
4a9b3be
@Roukoswarf
reuse existing code for tag query
66d1fdb
@Roukoswarf
fix patient specific tags
89557c1
@Roukoswarf
add more timestamp checks to TIDE params
5366e94
@Roukoswarf
partial handling of TIDE pagination, unfinished API
1b76577
@Roukoswarf
add tide pagination
12cb1b4
@Roukoswarf
fix some pointers in sprintf
5fd9f12
@Roukoswarf Roukoswarf changed the base branch from master to tide-v1 September 18, 2023 01:59
toddkazakov
toddkazakov reviewed Sep 19, 2023
View reviewed changes
Copy link
Contributor

@toddkazakov toddkazakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, this needs some unit tests. Everything else looks good.

Base automatically changed from tide-v1 to master October 26, 2023 17:04
@Roukoswarf Roukoswarf mentioned this pull request Nov 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toddkazakov toddkazakov toddkazakov left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Roukoswarf @toddkazakov