RADAR-base / RADAR-REST-Connector

A Kafka Source connector to receive data from REST APIs and publish them to Kafka. It has an extended version to support FitBit APIs.

https://radar-base.org/

Apache License 2.0

4 stars 4 forks source link

Release 0.5.2 #123

Closed mpgxvii closed 5 months ago

mpgxvii commented 6 months ago

Fixes Fitbit BR and Temperature converters (#121)
Fixes Fitbit HRV converter (#120)
Bumps radar-schemas, java to 17, and other deps (#122)
Update Fitbit HRV, BR, and Temperature routes to have a 30 day limit (#118)
Fix Fitbit SleepEfficiency converter (#116)
Adds Fitbit SpO2 route and converter (#119)

Bdegraaf1234 commented 6 months ago

Probably good to add a working codeql configuration, snyk workflow and perform security updates?

I will make PRs. If there is a hurry you can make this release first but it shouldn't take long

mpgxvii commented 6 months ago

Probably good to add a working codeql configuration, snyk workflow and perform security updates?

I will make PRs. If there is a hurry you can make this release first but it shouldn't take long

@Bdegraaf1234 Yes indeed would be good to have CodeQL working. Yes I can wait before making the release. Thank you!

Bdegraaf1234 commented 6 months ago

@mpgxvii Ready when you are ;)

124 <-- Ready to merge

125 <-- Needs snyk token in the repo secrets

126 <-- Should be merged after #125

Bdegraaf1234 commented 6 months ago

Why did the CodeQl action not run? 😆

Also maybe nice to rebase onto dev rather than merge to prevent unneeded merge commits

mpgxvii commented 6 months ago

Why did the CodeQl action not run? 😆

Also maybe nice to rebase onto dev rather than merge to prevent unneeded merge commits

@Bdegraaf1234 Ah yes will do. It didn't run because the branch was main instead of master. Updated it now on this release branch.

Bdegraaf1234 commented 6 months ago

I think this one is ready to go after rebasing the security updates #126 ^^