search

OCA / server-tools

Tools for Odoo Administrators to improve some technical features on Odoo.
https://odoo-community.org/psc-teams/tools-30
GNU Affero General Public License v3.0
705 stars 1.5k forks source link

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

Closed AungKoKoLin1997 closed 3 months ago

AungKoKoLin1997 commented 4 months ago

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

OCA-git-bot commented 4 months ago

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

OCA-git-bot commented 3 months ago

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 commented 3 months ago

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). 🤖

AungKoKoLin1997 commented 3 months ago

@kittiu Can you please review this changes?

AungKoKoLin1997 commented 3 months ago

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

pedrobaeza commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

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 commented 3 months ago

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