OCA / operating-unit

Manage Operating Units in Odoo
GNU Affero General Public License v3.0
72 stars 237 forks source link

[15.0][mig]-report_qweb_operating_unit #563

Closed JuanyDForgeflow closed 1 year ago

JuanyDForgeflow commented 1 year ago

@ForgeFlow

Removed clean and background templates since they're not on 15.0

Saran440 commented 1 year ago

Functional Test. I found error

Step to error

  1. Install module report_qweb_operating_unit and account
  2. Go to Settings > Companies > Configure Document Layout Selection_015
  3. It will error Selection_016
tiagor-exo commented 1 year ago

to fix this error you need to replace all the <t t-if="'operating_unit_id' in o.fields_get()">to <t t-if="o and 'operating_unit_id' in o.fields_get()">

AaronHForgeFlow commented 1 year ago

@JuanyDForgeflow , can you check?

JuanyDForgeflow commented 1 year ago

@tiagosrangel thanks! it works now @AaronHForgeFlow @Saran440 please try again

OCA-git-bot commented 1 year 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). 🤖

AaronHForgeFlow commented 1 year ago

/ocabot migration report_qweb_operating_unit /ocabot merge nobump

OCA-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-563-by-AaronHForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot commented 1 year ago

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