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 cli support for gerating Magic.Link authentication provider #644

Merged
merged 3 commits into from
Jun 2, 2020

Conversation

gizmoGremlin
Copy link
Contributor

@gizmoGremlin gizmoGremlin commented Jun 2, 2020

Closes #623

Generators: Add auth generator for Magic.Link

@gizmoGremlin
Copy link
Contributor Author

I'm noticing a typo in the commit message. Should I amend and force push or leave it?

@thedavidprice
Copy link
Contributor

@gizmoGremlin excited to see this one get started! re: commit message typo --> all good to leave as is.

Please let @cannikin know when this is ready for review. Thanks again!

@gizmoGremlin
Copy link
Contributor Author

@cannikin This is ready for review.

@cannikin
Copy link
Member

cannikin commented Jun 2, 2020

Seems simple enough! Have you used the generator and got the login actually working properly?

@gizmoGremlin
Copy link
Contributor Author

@cannikin Yes, I was able to create a magic.link signup in test mode using the magic.auth.loginWithMagicLink({ email }) method.

@cannikin
Copy link
Member

cannikin commented Jun 2, 2020

LET'S DO IT

@cannikin cannikin merged commit 31ff28c into redwoodjs:master Jun 2, 2020
@thedavidprice thedavidprice added this to the next release milestone Jun 2, 2020
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.

Generators: Add auth generator for Magic.Link
3 participants