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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Showcase login flow #1841

Merged
merged 1 commit into from Sep 1, 2023
Merged

Showcase login flow #1841

merged 1 commit into from Sep 1, 2023

Conversation

nmattia
Copy link
Collaborator

@nmattia nmattia commented Sep 1, 2023

This extracts the flow essence of the authenticateBox so that the flow can be showcased on /flows/loginManage.

In practice, most canister calls were abstracted away so that the flow can be used without an actual canister. Similar to the registration flow showcased in #1836.


馃煛 Some screens were changed

This extracts the flow essence of the authenticateBox so that the flow
can be showcased on `/flows/loginManage`.

In practice, most canister calls were abstracted away so that the flow
can be used without an actual canister. Similar to the registration flow
showcased in #1836.
Copy link
Member

@frederikrothenberger frederikrothenberger left a comment

Choose a reason for hiding this comment

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

Rubber stamp 馃憤

@nmattia nmattia added this pull request to the merge queue Sep 1, 2023
Merged via the queue into main with commit ffd75bb Sep 1, 2023
49 checks passed
@nmattia nmattia deleted the nm-extract-login-flow branch September 1, 2023 14:24
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.

None yet

2 participants