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

Feature/invite authentication #3

Merged
merged 57 commits into from
Apr 11, 2024
Merged

Conversation

nriedman
Copy link
Contributor

@nriedman nriedman commented Apr 10, 2024

Study Enrollment via Invitation Code Authentication and Customize Onboarding

♻️ Current situation & Problem

Currently, there is no way to validate the identity of the user of the app to enroll them in the study.

⚙️ Release Notes

  • Customized Spezi Onboarding Flow to have language and symbols relevant to Engage
  • Integrated invitation code authorization into the onboarding flow, using cloud functions to validate supplied code against a database of valid codes stored in Firebase.

📚 Documentation

Integrated code based on:
StanfordBDHG/PediatricAppleWatchStudy#54.

The Onboarding flow now has custom views:

Screenshot 2024-04-09 at 10 24 14 PM Screenshot 2024-04-09 at 10 26 21 PM Screenshot 2024-04-09 at 10 26 28 PM

See SpeziOnboarding for documentation of the other views.

✅ Testing

Thoroughly tested the onboarding flow, including the invitation code authorization and account creation process.

Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

nriedman and others added 30 commits April 3, 2024 13:35
Removed the following dependencies:
SpeziContact
SpeziScheduler
SpeziMockWebService

Removed filler tabs on landing page, leaving only Home.
@nriedman nriedman requested a review from PSchmiedmayer April 10, 2024 05:44
Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job with the PR @nriedman! Great to see the PAWS code in this project and the invitation code flow working! I only had minor comments and suggestions, feel free to merge the PR after these have been addressed and the UI tests are passing without any issues 🚀

@nriedman nriedman enabled auto-merge (squash) April 11, 2024 21:26
@PSchmiedmayer PSchmiedmayer disabled auto-merge April 11, 2024 22:11
@PSchmiedmayer PSchmiedmayer merged commit 15b47f6 into main Apr 11, 2024
5 checks passed
@PSchmiedmayer PSchmiedmayer deleted the feature/inviteAuthentication branch April 11, 2024 22:11
@nriedman nriedman restored the feature/inviteAuthentication branch April 11, 2024 23:12
@nriedman nriedman deleted the feature/inviteAuthentication branch April 11, 2024 23:12
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