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

Rename completion to callback and simplify actor usage. #6141

Merged
merged 1 commit into from
May 11, 2022

Conversation

pixlwave
Copy link
Member

@pixlwave pixlwave commented May 10, 2022

A small tidy up following on from #6056 and #6048.

  • Completion handlers that are called more than once in the onboarding flow are renamed to callback.
  • Actor usage is simplified in start and process(viewAction:) to align with Email/Terms/ReCaptcha screens.

@pixlwave pixlwave marked this pull request as ready for review May 10, 2022 16:38
@pixlwave pixlwave requested review from a team, phloux, MaximeEvrard42 and stefanceriu and removed request for a team and phloux May 10, 2022 16:38
@github-actions
Copy link

📱 Scan the QR code below to install the build for this PR.
🔒 This build is for internal testing purpose. Only devices listed in the ad-hoc provisioning profile can install Element Alpha.

QR code

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/YWpv2P

@pixlwave
Copy link
Member Author

(The force push resolved merge conflicts, nothing changed).

@pixlwave pixlwave merged commit 835cdda into develop May 11, 2022
@pixlwave pixlwave deleted the doug/onboarding_callback branch May 11, 2022 12:18
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.

3 participants