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

Feature/user authentication - Merge history #10

Closed
wants to merge 12 commits into from

Conversation

sjeguedes
Copy link
Owner

@sjeguedes sjeguedes commented Nov 11, 2019

Merge was needed after rewriting history.

This branch treated expected changes and "fixed" issue: #7

…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
…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.
@sjeguedes
Copy link
Owner Author

Merge need to be made with "develop" branch and then "master" branch will be rebased with "develop".

@sjeguedes sjeguedes closed this Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant