server-tools icon indicating copy to clipboard operation
server-tools copied to clipboard

[17.0][REF] auditlog: fetch field data as user

Open gbrito opened this issue 1 year ago • 6 comments

Currently auditlog fetch field data as sudo It doesn't make sense to use sudo to fetch the data since the user only have access to edit fields that he has access too. By using sudo we bypass multi company rules

This commit will fix #2554

gbrito avatar Nov 08 '24 09:11 gbrito

@NL66278 @andreschenkels Can you guys make a functional test on it, as far as I see it works as expected and didn't broke any functionality

gbrito avatar Nov 08 '24 10:11 gbrito

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar Nov 13 '24 09:11 OCA-git-bot

@gbrito The PR is actually for 17.0, not for 18.0 Please update the description. Also needs rebase.

NL66278 avatar Jul 07 '25 08:07 NL66278

@NL66278 description was my bad, fixed now.

StefanRijnhart avatar Jul 07 '25 08:07 StefanRijnhart

This resolved https://github.com/OCA/server-tools/issues/2554 in our case.

n17 avatar Aug 01 '25 08:08 n17

#2554 should be solved now with https://github.com/OCA/server-tools/pull/3373. Does your change have additional value that you want to rebase or can we close this PR?

StefanRijnhart avatar Oct 01 '25 12:10 StefanRijnhart