server-tools icon indicating copy to clipboard operation
server-tools copied to clipboard
verified publisher icon OCA

[16.0][FIX] excel_import_export: remove sudo() when importing record

Open AungKoKoLin1997 opened this issue 1 year ago • 4 comments

This commit fixes issue of finding multiple records across all companies during import when they have same record name. Previously, importing with sudo could bypass multi-company record rules.

@qrtl QT4715

AungKoKoLin1997 avatar Jul 19 '24 04:07 AungKoKoLin1997

Hi @kittiu, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Jul 19 '24 04:07 OCA-git-bot

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 Jul 31 '24 04:07 OCA-git-bot

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 Jul 31 '24 04:07 OCA-git-bot

@kittiu Can you please review this changes?

AungKoKoLin1997 avatar Jul 31 '24 15:07 AungKoKoLin1997

@oca/server-tools-maintainers Can we get this PR merged?

AungKoKoLin1997 avatar Aug 19 '24 03:08 AungKoKoLin1997

/ocabot merge patch

pedrobaeza avatar Aug 19 '24 06:08 pedrobaeza

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-2992-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot avatar Aug 19 '24 06:08 OCA-git-bot

Congratulations, your PR was merged at 7516bd09944c2551fd01df1ef7cc4615009b82b4. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Aug 19 '24 06:08 OCA-git-bot