[14.0][ADD] impersonate_login #637
Conversation
Kev-Roche
force-pushed
the
14.0-add-impersonate_login
branch
from
e7334b8
to
9cfe6f4
Compare
Kev-Roche
force-pushed
the
14.0-add-impersonate_login
branch
from
9cfe6f4
to
84a704f
Compare
|
@Kev-Roche Did you already forward port this to v16/v17? |
| comodel_name="res.partner", | ||
| string="Impersonated Author", | ||
| compute="_compute_impersonated_author_id", | ||
| store=True, |
There was a problem hiding this comment.
This will take forever to compute in active databases.
There was a problem hiding this comment.
Maybe just ALTER TABLE mail_message ADD COLUMN impersonated_author_id INTEGER;
|
@Kev-Roche We are working on a v17 port. Like the module. |
Hi @CasVissers-360ERP , thanks for your interest. I did not work on other versions yet because tests are missing and I m not sure if the module is working properly. |
|
Well the module will make our lives a lot easier. I will ask more team members to review. |
| request.session.session_token = security.compute_session_token( | ||
| request.session, request.env | ||
| ) | ||
| return { |
There was a problem hiding this comment.
If we go to a user without access to the current URL we get an access error, might be better to redirect to main menu?
|
@Kev-Roche Please feel free to include some of the changes I did there, for example the revised |
Kev-Roche
force-pushed
the
14.0-add-impersonate_login
branch
7 times, most recently
from
6218882
to
9c45742
Compare
Kev-Roche
force-pushed
the
14.0-add-impersonate_login
branch
from
9c45742
to
a108a9a
Compare
|
@CasVissers-360ERP , @astirpe thank you both for your time and help on this module, I bring some of your changes from V16/17 here. |
This module allows to login as another user, its purpose is mainly (only?) for testing and the support team.
As an alternative to auth_admin_passkey, for a more French/European data security complience.
No tests wrote in this module yet, any hints / help will be appreciated.
After adding tests, I will migrate the module in v16.