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

Bugfix/KBDEV-1200-ClinicalTrialsgov-loaded-records-repeatedly #150

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Bugfix/KBDEV-1200-ClinicalTrialsgov-loaded-records-repeatedly #150

merged 1 commit into from
Apr 18, 2024

Conversation

sshugsc
Copy link
Contributor

@sshugsc sshugsc commented Apr 16, 2024

KBDEV-1200

  • fix pageToken error

@sshugsc sshugsc self-assigned this Apr 16, 2024
@mathieulemieux mathieulemieux requested a review from elewis2 April 16, 2024 20:28
@mathieulemieux
Copy link
Member

@elewis2, didn't wanted to re-request your review, sorry! Would prefer in fact we wait until @sshugsc is done with testing.

@sshugsc
Copy link
Contributor Author

sshugsc commented Apr 16, 2024

Convert to a draft PR for now, and wait for running on staging tonight.

@sshugsc sshugsc marked this pull request as draft April 16, 2024 20:44
@sshugsc sshugsc marked this pull request as ready for review April 17, 2024 16:32
@sshugsc
Copy link
Contributor Author

sshugsc commented Apr 17, 2024

Got the logs running on staging clinicaltrialsgov_all_staging.log.zip

@sshugsc sshugsc merged commit 27511ed into develop Apr 18, 2024
5 checks passed
@github-actions GitHub Actions
Copy link

Unit Test Results

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 27511ed. ± Comparison against base commit dee7eda.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mathieulemieux mathieulemieux Awaiting requested review from mathieulemieux

@elewis2 elewis2 Awaiting requested review from elewis2

Assignees

@sshugsc sshugsc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sshugsc @mathieulemieux @elewis2