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

Small content adaptions based on Sinai feedback #588

Closed
10 tasks done
tamslo opened this issue Apr 12, 2023 · 0 comments · Fixed by #600
Closed
10 tasks done

Small content adaptions based on Sinai feedback #588

tamslo opened this issue Apr 12, 2023 · 0 comments · Fixed by #600
Assignees
Labels
P: App Project: App T: Change Type: Changes

Comments

@tamslo
Copy link
Collaborator

tamslo commented Apr 12, 2023

  • Include onboarding again
  • Change order of onboarding and login
  • Adapt text
    • Onboarding screen 1: change exclamation mark to a period; add a blank line between that line and the next sentence; spell out the word “percent” per our style
    • Onboarding screen 2: instead of “Your body reacts to drugs uniquely,” say, “Each person’s body reacts to drugs differently.” (The way it is now comes across as a little snarky.)
    • Onboarding screen 3: make it say “PharMe warns you if your genome makes you more likely to experience an unintended drug response. This enables you to avoid drugs that are ineffective or have side effects.” (note “side effects” does not take a hyphen
    • FAQ screen: “pharmacogenomics” should be lowercase
  • Make sure logic when login is shown and when not (data is not there) works ➡ current logic using MetaData.instance.isLoggedIn working; is only set if data downloaded and unset if data deleted
  • Change onboarding to (meaningful) placeholder for tutorial ➡️ just keep onboarding as it is, can be adapted in Add app tutorial #259
  • The language looks generally clean. I would suggest, starting on screen 14, that you change "I am actively taking this drug" to "I am currently taking this drug." Also on screen 14, make it "slower than expected" not "slower as expected" in the alert section.
@tamslo tamslo self-assigned this Apr 12, 2023
@tamslo tamslo converted this from a draft issue Apr 12, 2023
tamslo added a commit that referenced this issue Apr 18, 2023
tamslo added a commit that referenced this issue Apr 18, 2023
@tamslo tamslo mentioned this issue Apr 26, 2023
@tamslo tamslo added P: App Project: App T: Change Type: Changes labels Apr 27, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done (last 14 days) in PharMe Study Preparation Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P: App Project: App T: Change Type: Changes
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant