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

Administrator email management and features #1907

Closed
wants to merge 60 commits into
base: master
from

Conversation

Projects
None yet
5 participants
@roadster31
Copy link
Contributor

roadster31 commented Jan 8, 2016

This PR is a fix for #1510. It adds an address email to the administrator profile. This address email can now be used to login just like the login name.

An administrator could now recover a lost password, just like a regular customer.

Minor improvement: add a language selector on back-office login page.

@roadster31 roadster31 changed the title [WIP] Admin email Admin email Jan 9, 2016

@roadster31 roadster31 changed the title Admin email Administrator email management and features Jan 10, 2016

@bibich

This comment has been minimized.

Copy link
Contributor

bibich commented Jan 13, 2016

Nice 👍
It would be great to add an option to deactivate the lost password feature.
And please could you generate the SQL for translated languages : php Thelia generate:sql --locales='de_DE,en_US,es_ES,fr_FR'

@roadster31

This comment has been minimized.

Copy link
Contributor Author

roadster31 commented Jan 14, 2016

The lost password feature has been added.

@roadster31 roadster31 force-pushed the roadster31:admin-email branch from ed18c89 to 75a83f3 Jan 20, 2016

@roadster31 roadster31 closed this Jan 27, 2016

@roadster31 roadster31 deleted the roadster31:admin-email branch Jan 27, 2016

@roadster31 roadster31 restored the roadster31:admin-email branch Jan 27, 2016

@roadster31 roadster31 reopened this Jan 27, 2016

@roadster31 roadster31 force-pushed the roadster31:admin-email branch 2 times, most recently from 6e3c989 to 26987a1 Jan 27, 2016

@roadster31

This comment has been minimized.

Copy link
Contributor Author

roadster31 commented Feb 2, 2016

If an admin logs in the BO without a valid email address, he's redirected to its profile, and incited to enter an email address.

img-2016-02-02 13 41 02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment