revelry-foundation / Project-Riddell

All Project-Riddell issue-tracking + additional, non-repo-specific docs and links.
https://nightingale.revelry.org/
MIT License
0 stars 0 forks source link

Positive User - COVID-19 Status - Share Locations Based on Onset of Symptoms #46

Open iamjoshfrank opened 4 years ago

iamjoshfrank commented 4 years ago

This Issue is a future iteration of #50.

Background

Share my location isn't needed until a user indicates they have tested positive. We currently have no use / no desire to gather the location data of negative/unknown-status users.

Flow should look something like this for sharing location history to the server:

  1. Change status to positive
  2. Supply additional information (dates for test / test result / start of symptoms)
  3. Request the user agree to share locations for the relevant time period they were likely contagious

Wireframes for Reference

Please see #47 for guidance on layout and userflow.

iamjoshfrank commented 4 years ago

"Relevant time period" should equal the contact elicitation period, eg the period of time the positive-status user was likely infectious. See my comment on #44, I'm not sure where this should land. Probably here, since we don't want to upload a bunch of irrelevant location history - only the list of locations that fall within the infectious period.

Update: added the CDC guidance on the contact elicitation period in this comment on #42.

CurtisVonRubenhoff commented 4 years ago

I think this ticket falls under the purview of what I'm working on (API endpoint to make this possible). I self-assigned this while I'm working, but it won't be done until it's implemented in the mobile app (maybe i can shift my focus to that unless someone currently on the mobile side is itching to grab this).

I should have a pr for the API endpoint within the hour.

iamjoshfrank commented 4 years ago

Awesome, thanks @CurtisVonRubenhoff !

I'll be back this way in ~20 minutes to finish off making the Scenario and Acceptance Criteria crystal clear re: the date range for which the app should share locations based on test and/or symptom dates.

I'm assuming that work (that logic?) would happen on the mobile end; and that the API only serves as a landing spot for the selected location data. Yeah?

CurtisVonRubenhoff commented 4 years ago

That's accurate. This endpoint is only ingesting locations and when you were there

Mcscott142 commented 4 years ago

Wires:

Onboarding:

Screen Shot 2020-05-28 at 5 56 33 PM Screen Shot 2020-05-29 at 9 53 44 AM Screen Shot 2020-05-29 at 10 08 44 AM
iamjoshfrank commented 4 years ago

Update! Based on conversations in Slack spurred by @grossvogel here and here re: trade-offs and decisions when we're trying to ship a Proof of Concept in under a week:

  1. We're going to postpone the more complicated (but much more accurate) UI of entering dates for test results and onset of symptoms in order to filter down location dates for submission.
  2. For now, we'll upload the prior 14 days of locations from the date the user changes their status from "negative" to "positive" (still privately, anonymously, and only with their discrete consent).

Spinning up a clean Issue for ^^ and I'm going to move this Issue along with #42 into a future Milestone.

iamjoshfrank commented 4 years ago

@CurtisVonRubenhoff - see updated comment ^^

Moving #50 forward as the "today solution."

Does this mess things up for you?

CurtisVonRubenhoff commented 4 years ago

Nope. I think I'm still good. I'll move my updates to #50 in that case though