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.

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

Fitbit Subscription API #52

Open
wants to merge 34 commits into
base: dev
Choose a base branch
from
Open

Conversation

jzhou59
Copy link

@jzhou59 jzhou59 commented Nov 5, 2022

Integrate Fitbit subscription API into RADAR-PushEndpoint.
Mainly modules:

  • Create subscriptions for users to receive notifications.
  • Create a service to receive notifications and store them(actually the offset of scope) in Redis.
  • Create a processor to retrieve data from Fitbit in a real-time way.

@yatharthranjan
Copy link
Member

I think you can change the base to dev

@yatharthranjan
Copy link
Member

Just elaborating on the things to be done:

  1. Test data received directly from fitbit and successful upload to kafka (also check redis impl is working as expected)
  2. Extracting common code between rest-connector and push-endpoint to libraries so can be reused. This would include code for requests and routes primarily.
  3. Documentation of the code and also in the README (to add how to use the fitbit subscription including configuration on fitbit portal)

Number 2 can be tackled later by adding a github issue for it.

@yatharthranjan yatharthranjan changed the base branch from fitbit_updates to dev March 5, 2024 17:30
@yatharthranjan yatharthranjan mentioned this pull request Mar 8, 2024
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants