-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Release 0.5.3 #142
Release 0.5.3 #142
Conversation
mpgxvii
commented
Jul 10, 2024
- Update Oura connector request back off times (Update Oura request backoff times #134)
- Update Oura connector offset initialisation (Fix Oura offset issues #141)
- Fix Fitbit connector pulling of users (Fix get users #139)
- Update connectors Dockerfile to use Java 17 to fix build (Uninstall java 11 and install java 17 in the base docker images #138)
Add snyk workflow to github actions
Security updates
Sync release with dev
Update Oura request backoff times
Uninstall java 11 and install java 17 in the base docker images
Fix get users
Fix Oura offset issues
ouraOffsetManager.updateOffsets( | ||
request.route, | ||
request.user, | ||
request.endDate, | ||
) | ||
userNextRequest[request.user.versionedId] = Instant.now().plus(BACK_OFF_TIME) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update to ONE_DAY
Co-authored-by: Yatharth Ranjan <yatharthranjan89@gmail.com>
Allow null end dates in Oura
LGTM, under the assumption that most of this code was already reviewed in PRs I just looked at the recent changes in separate commits |
Fix isComplete check for User
@yatharthranjan @Bdegraaf1234 Just doing some final testing then will release. |
oura-library/src/main/kotlin/org/radarbase/oura/request/OuraRequestGenerator.kt
Outdated
Show resolved
Hide resolved
Fix Oura offset times
can we fix the vulerability before the release? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Can you upgrade to the following versions for vulnerability fixes-
|
Done, thanks! |
great, LGTM! |
We should add PRs to |