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

Add UI callback logging #125

Merged
merged 2 commits into from
Nov 18, 2022
Merged

Add UI callback logging #125

merged 2 commits into from
Nov 18, 2022

Conversation

chris-stytch
Copy link
Contributor

Description

This PR adds console logging for UI callbacks, i.e. onEvent and onError.

This also shrinks the code snippets a bit for better fit.

@vercel
Copy link

vercel bot commented Nov 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
stytch-nextjs-integration ✅ Ready (Inspect) Visit Preview Nov 18, 2022 at 9:37PM (UTC)

@@ -137,29 +128,31 @@ const trigger = useCallback(async () => {
details:
'Build an email/password authentication experience including passwords resets, password strength checking, and magic links using prebuilt Stytch UI components.',
description: ``,
instructions: `To the right you'll the Stytch UI configured for email/password login. Enter a new email address and you will be prompted to create an account with a secure password.`,
instructions: `To the right you'll the Stytch UI configured for password login. Enter a new email address and you will be prompted to create an account with a secure password.`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Noticed we are missing a verb

"to the right you'll ???..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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