10-07-2017, 09:05 PM
When you remove the data in the username field it goes to "Unable to check"...we should just remove the message if the field is empty. We should also add this for email. And I would like to deploy this to admin_users.php for the manual add please.
The glogin select is messed up. It always says enabled. I committed a change for this.
What are we doing about more graceful token fails? Do you want this in 43 or is this US5?
Should we turn message notifications on by default? Or off? Do users setup their email settings during setup? If we enable it by default we could run into issues with email failures.
I committed a change to add the manage navigation button to admin.php.
Also: the Help menu in the DB-driven nav is set to logged in only, set this to no, we don't want logged in users to see it, and don't want guests to not have this menu. You also need to add forgot password to this menu.
The glogin select is messed up. It always says enabled. I committed a change for this.
What are we doing about more graceful token fails? Do you want this in 43 or is this US5?
Should we turn message notifications on by default? Or off? Do users setup their email settings during setup? If we enable it by default we could run into issues with email failures.
I committed a change to add the manage navigation button to admin.php.
Also: the Help menu in the DB-driven nav is set to logged in only, set this to no, we don't want logged in users to see it, and don't want guests to not have this menu. You also need to add forgot password to this menu.