-
Notifications
You must be signed in to change notification settings - Fork 806
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
Confirming 2FA requires additional changues to an external config file #1025
Comments
Yeah, you definitely have to do this manually if you want to enable 2FA confirmation. Normally nothing should break though. I can't immediately see this from your explanation but do things break when you simply run composer update? And if so: what error do you get? |
Thoughts on the having to edit the fortify config file or any file that needs manual edits maybe needs to be documented in the UPGRADING.md file as you should not really need to dig through the source code to upgrade between 2.x (MINOR) versions? (I miss the recipes:update from Symfony) I just upgraded an old Jetstream 2.6 project and I'm unable to reproduce any errors or unexpected behaviour without the |
Right. I don't think anything will break if you run a composer update. Listing all upgrade steps between minor versions is an impossible task as there's way too much files involved. It basically boils down to copying the changes from the stubs. Remember that Jetstream is a scaffold and there's no easy way to upgrade to newer functionality in stubs. People might have edited their stubs already so it's much more complicated than just copy/paste. |
Description:
Well me again, I'm sorry. Confirming 2FA #992 it's a nice and really appreciated edition problem is, if we do a simple
composer update
well it doesn't kind of work because we have to add'confirm' => true
to the fortify config file. it's not a big of a problem (once you found out why the newly addition doesn't work. I'm ashame to admint it took longer than i wished to figure it out) to add a simple line it's not a big deal, the thing is when you do acomposer update
i believe you are expected to have things broke on the updates of the package you require, but we don't explicitly require 'laravel/fortify' it's required within the package and the new'laravel/fortify'
version adds that line laravel/fortify@0fa359a so, i stil thing it should bea breaking changue update but it's here now, and for the people that don't know why this newly addition isn't working, maybe try and see if you have that line.Steps To Reproduce:
Install previous jetstream version (2.6.8) do a composer update, and 2FA isn't really working
The text was updated successfully, but these errors were encountered: