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

Andy/227 finalizing #228

Merged
merged 3 commits into from
Dec 12, 2024
Merged

Andy/227 finalizing #228

merged 3 commits into from
Dec 12, 2024

Conversation

andyvo2004
Copy link
Contributor

@andyvo2004 andyvo2004 commented Nov 17, 2024

227

Issue Number(s): #227

What does this PR change and why?
add keyboard-aware view to personal info page so user can see what they enter, fix date validation for personal info page, use edit-patient api for profile screen (untested)

Checklist

  • Task 1
  • Task 2
  • App tested and bugs fixed

Critical Changes / Notes

  • Api is untested, so not sure if updating user info works

Related PRs

Testing

test personal info page and ensure everything can be seen when entering info, ensure invalid dates cannot be used in personal info page, test profile page to see if info is updated with the api

Copy link

🚀 Expo preview is ready!

  • Project → brain-exercise
  • Platforms → android, ios
  • Scheme → myscheme
  • Runtime Version → 1.0.0
  • More info

Learn more about 𝝠 Expo Github Action

@johannesq23 johannesq23 merged commit 008263c into develop Dec 12, 2024
2 checks passed
@johannesq23 johannesq23 deleted the andy/227-finalizing branch December 12, 2024 20:01
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