search

OCA / rma

Odoo for Return Merchandise Authorization (RMA)
GNU Affero General Public License v3.0
78 stars 206 forks source link

[14.0] [FIX] uom_id in rma and rma_sale_mrp reports #387

Closed raumschmiede-joshuaL closed 3 months ago

raumschmiede-joshuaL commented 3 months ago

Field name is wrong in report. uom_id doesn't exist in model rma.

Raised an error in our production system, we fixed it by manually changing the field in the report via frontend.

@mt-software-de

OCA-git-bot commented 3 months ago

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

mt-software-de commented 3 months ago

Thx. Could you please fix it also there? https://github.com/OCA/rma/blob/14.0/rma%2Fviews%2Freport_rma.xml#L113

image

OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-387-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 3 months ago

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

raumschmiede-joshuaL commented 3 months ago

@pedrobaeza, for upper versions somebody else has to do it if it is needed soon. I can do it later if I have more time in the future. @mt-software-de, do you want to do this earlier?

pedrobaeza commented 3 months ago

Well, I don't think it's too much work, and you should provide the whole fw-port chain for avoiding unsynced branches. Be generous on this for receiving the same from others.

raumschmiede-joshuaL commented 3 months ago

Ok, I will do. Currently I have time, I have to wait for new tickets :D

raumschmiede-joshuaL commented 3 months ago

Done. @mt-software-de fyi

pedrobaeza commented 3 months ago

Thank you!