Skip to content

[17.0][FW] base_technical_user: port from 14.0 - #3094

Open
Kimkhoi3010 wants to merge 3 commits into
OCA:17.0from
Kimkhoi3010:oca-port-base_technical_user-14.0-to-17.0-54efc4
Open

Kimkhoi3010 wants to merge 3 commits into
OCA:17.0from
Kimkhoi3010:oca-port-base_technical_user-14.0-to-17.0-54efc4

Conversation

@Kimkhoi3010

@Kimkhoi3010 Kimkhoi3010 commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

rousseldenis and others added 3 commits October 11, 2024 18:25
First, the company should be taken through self.env.company as user.company_id
is now the default company for the user (and not the current one).

Secondly, the actual code does not allow to do recordset.with_company(B).sudo_tech()....
as allowed_company_ids in context can be filled in with current company
(and not the one of the tech user - which is unique)
When we use use_tech(raise_if_missing=True) and the technical user (user_tech_id) is not defined on the current company, the error message who said that field is empty should be related to the active company and not related to the default user's company.
@sebalix

sebalix commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

@Kimkhoi3010 I updated your PR description to put the links of ported PRs

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.

5 participants