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

Sync BP data with server #17

Closed
7 tasks done
ssrihari opened this issue May 16, 2018 · 0 comments
Closed
7 tasks done

Sync BP data with server #17

ssrihari opened this issue May 16, 2018 · 0 comments

Comments

@ssrihari
Copy link
Contributor

ssrihari commented May 16, 2018

TODO:

  • Finalise data model for blood pressure
  • Add model, migrations, same validations as other models
  • Include mergeable concern, add mergeable test for BP
  • Write specs for blood pressure api
  • Implement sync: adding blood pressures, test this
  • Implement sync: retrieving blood pressures, test this

Questions:

  • Do we compute risk level on mobile, store it, send it to server?
    • Assumption: we'll compute it on mobile, not send it to server

Data Model, API spec

  • id: uuid
  • systolic: integer
  • diastolic: integer
  • created_at: timestamp
  • updated_at: timestamp
  • patient_id: uuid
  • all are required
@ssrihari ssrihari added this to This Iteration in RedApp May 16, 2018
@govindkrjoshi govindkrjoshi moved this from This Iteration to Doing in RedApp May 29, 2018
@ssrihari ssrihari assigned saket and unassigned ssrihari and govindkrjoshi Jun 1, 2018
@saket saket moved this from Doing to PR Review, QA in RedApp Jun 5, 2018
@saket saket mentioned this issue Jun 5, 2018
@saket saket moved this from PR Review, QA to Done in RedApp Jun 5, 2018
@timcheadle timcheadle removed this from Dev+QA Complete in RedApp Jul 10, 2018
@saket saket closed this as completed Nov 8, 2018
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

No branches or pull requests

4 participants