Feature/user authentication #11
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ted cascade and orphanRemoval Doctrine annotation attributes to decide later what to do during actions for persist, remove, etc
…. Fixed other entities to strenghten assertion for updateDate in constructor
…face on UserRepository. Checked HTTP basic authentication to validate security configuration. Improved other entities. Unit testing needs to be enlarged for passed and future functionalities to improve project stability
…gout - flash messages
…ed remember me functionality with cookie lifetime in framework.yaml - fixed main role label on header user profile
…nal access to form with user id and token and renewal confirmation. Need to continue refactoring by modifying services to make actions lighter - entity service layer UserManager and form handlers.
…ch as creating form handlers to manage request for each form
…with ParameterBagInterface for dependency injection in service classes. Improved custom form errors in Twig templates.
…lers to manage validations (constraints and custom validation) and call final actions (user update, email sending...), email configuration to feed the mailer and custom event instances created with factories.
…fications, buttons.
|
Merge accepted with comments: Check untracked files more regularly. Pull "develop" origin branch on local branch to synchronise local and remote repositories. Don't forget to rebase "master" branch with commits created on "develop" branch when being on local repository. Then create a tag for rebase operation. Push "master" branch and tag on remote repository. |
|
Please note that:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This branch creates authentication forms process (login, password renewal request and reset pages).
This merge aims at testing better workflow with development and features branches added to master. No issue is referenced for this pull request.
This branch treated expected changes and "fixed" issue: #7
Added login (with remember me functionality) and logout action (with event handler)
Added authentication checker to verify account activation and control login access
Started first unit tests on User entity
Added password renewal request (unique access with personal link provided by email) and authorization
Added password renewal (with confirmation by email)
Refactored actions (controllers) scripts by using form handlers, email configuration to pass to mailer, custom event, interfaces and factories
These pages need to be completed with User registration and account validation process.
Entities constructors improvements are also necessary.
Message: please accept this merge and be conscious multiple changes will be made later.