-
Notifications
You must be signed in to change notification settings - Fork 136
For a new CI4 project #2
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
Comments
It is not anywhere close to complete. But feel free to keep an eye on it :) |
Hi @lonnieezell ;-) how are you? |
I'm using it as I'm building out Bonfire and I think it's 99% there for production. There's one issue a user has reported with the "actions" (after login/registration optionals like 2FA and email verification. Those currently rely on it being in the session which probably not the right way to go with that. Other than that I would call the code production ready. Mostly what I have left on it is documention and see if I can make the onboarding process smoother with it. |
Thank you. It looks like gr8 piece of code. |
May you guys share Events config somehow? |
There's no configuration needed if you don't need to use it. If you do, just add it to your Events::on('didLogin', function (...$params)
{
// do stuff here
}); |
Ohh, that's ok. I was not sure if those events should fire something by default or not. About:
What do you think about idea to simplify code in Register/LoginController: It is:
it could be:
and check class that should support it directly in ActionController? BTW/ I would name those routes as you did with "verify-magc-link"; |
Hi Lonnie,
Is this repo a replacement for Myth-auth?
I am starting a new CI4 project, can this package be use instead of myth-auth.
Thank you
The text was updated successfully, but these errors were encountered: