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

Integrate FOSUserBundle (for registration, password-reset, etc.) #144

Closed
tacman opened this issue Apr 29, 2018 · 5 comments · Fixed by #216
Closed

Integrate FOSUserBundle (for registration, password-reset, etc.) #144

tacman opened this issue Apr 29, 2018 · 5 comments · Fixed by #216
Milestone

Comments

@tacman
Copy link

tacman commented Apr 29, 2018

Description

This is a great example of Symfony4 app with the AdminLTE theme. But there's no way to register or reset a forgotten password.

Possible Fix

Integrate FOSUserBundle.

@kevinpapst
Copy link
Member

Hey @tacman, thanks for your feedback.

Right now "no register" is on purpose, as only admins should be able to create new user.
Resetting a password is indeed something different. I'll look into FOSUserBundle.

@kevinpapst
Copy link
Member

kevinpapst commented Jun 13, 2018

After having some deeper looks into FOSUserBundle I decided against going with it, but instead going to code it myself, see #163 and #164.

There are several reasons why I am not going to use it, probably the most important one is missing core compatibility with Symfony 4.1. Also there are quite some good write-ups why this bundle is not the best solution for every use-case.

I am closing this ticket in favor of the new ones which are linked above.

@kevinpapst kevinpapst added this to the 0.2 milestone Jun 23, 2018
@kevinpapst
Copy link
Member

Reopen again. After quite a lot of testing I figured that it will be easier to integrate this bundle instead of building it with another solution.

@kevinpapst kevinpapst reopened this Jul 14, 2018
@kevinpapst kevinpapst modified the milestones: 0.2, 0.3 Jul 14, 2018
@kevinpapst kevinpapst mentioned this issue Jul 14, 2018
19 tasks
@kevinpapst
Copy link
Member

Thanks for the inspiration @tacman
FOSUserbundle is now integrated in the latest master

@lock
Copy link

lock bot commented Nov 18, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Nov 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants