lonnieezell / Bonfire2

CodeIgniter 4-based application skeleton
MIT License
127 stars 49 forks source link

Fixed Auth Config $views['layout'] value... #436

Closed daracorcoran closed 4 months ago

daracorcoran commented 4 months ago

...to reference the appropriate Shield layout, instead of 'master'.

Same issue as #352

Was getting an Invalid File error after login, on a fresh install, with 2FA enabled. Looks like the value should be the Shield view layout?

dgvirtual commented 4 months ago

Have you checked how it actually works? Shield layouts are not used in Bonfire2, since there is a different theming mechanism here.

daracorcoran commented 4 months ago

You're right, I forgot there is a separate Auth theme. I'll close the PR for now, it'll need more work. But the error persists.

Looks like Bonfire is using the Shield controller for 2FA so it's bypassing the themeable controllers. Bonfire might need to overwrite those routes as it does for login & register etc. in Auth > Config > Routes