Fixed Register
Fixes
Fixes for #310 (closed) by @Karna98
Description
After merging !134 (merged), UserTable.php was modified which break the register process.
Type of PR
This PR is an improvement.
Tests
-
In development mode, (by default)
2FA will not be enforced and you can skip the 2FA setup or verify process.
Try Registering new user and log in. -
In production mode, (In .env, change APP_ENV = 'Production') this will allow you to test 2FA which will be used in production.
A. Register New User. B. Log in with the new user.
Try to mess up registration or login by not setting/verifying up 2FA or opening a new link (related to CDLI to bypass 2FA) on another tab.
-
Check if Forgot Password functionality is working as expected.
Checklist:
-
My pull request has a descriptive title (not a vague title like "Update index.md
"). -
My pull request targets the phoenix/develop
branch of the repository. -
My commit messages follow best practices. -
My code follows the established code style of the repository. -
I tried running the project locally and verified that there are no visible errors.