Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] util.indirect_references: new field iap_extracted_words #182

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RicGR98
Copy link

@RicGR98 RicGR98 commented Jan 3, 2025

IAP extracted words are now no longer only linked to account.move's but can be linked to any model using res_model and res_id.

@robodoo
Copy link
Contributor

robodoo commented Jan 3, 2025

Pull request status dashboard

@RicGR98 RicGR98 force-pushed the master-manual-correction-ocr-bank-stmt-rigr branch from 57e58d1 to aacde52 Compare January 8, 2025 10:42
@KangOl
Copy link
Contributor

KangOl commented Jan 8, 2025

Can you please refer to correct enterprise PR in the commit the message?

@RicGR98 RicGR98 force-pushed the master-manual-correction-ocr-bank-stmt-rigr branch from aacde52 to fc289a0 Compare January 8, 2025 15:11
@RicGR98
Copy link
Author

RicGR98 commented Jan 8, 2025

Can you please refer to correct enterprise PR in the commit the message?

Fixed :)

Still puzzled by why upgradeci/matt has an error while Runbot's upgrade ci don't 🧐
Do you have any lead ?

@aj-fuentes
Copy link
Contributor

Still puzzled by why upgradeci/matt has an error while Runbot's upgrade ci don't 🧐

Because there are missing forward ports most probably. Did you ever wonder what the runbot exceptions when adding a field or view in a stable version was about? Well, it was to hide the errors. But the message is still there:

 Field unlink res.users.hide_peppol_fields explicitly ignored, skipping 

Also, upgradeci tests many more version combinations than just the major ones as runbot upgrade CI does ;)

@RicGR98 RicGR98 force-pushed the master-manual-correction-ocr-bank-stmt-rigr branch from fc289a0 to 2c8ecb7 Compare January 9, 2025 09:35
@RicGR98
Copy link
Author

RicGR98 commented Jan 9, 2025

@aj-fuentes

Thanks for the explanation ! If I understood correctly there is nothing I should regarding hide_peppol_fields but I'm still having other errors that I do not have while upgrading my db locally (https://upgradeci.odoo.com/upgradeci/run/180639) which are totally related to this 🤔

I've refixed the upgradeci/pre-commit in the meantime

@aj-fuentes
Copy link
Contributor

Ah OK. That's because you need to link here your upgrade PR: https://github.com/odoo/upgrade/pull/7010

@aj-fuentes
Copy link
Contributor

upgradeci retry

@aj-fuentes
Copy link
Contributor

Nope, retry won't work. You need to put the linked PRs in the commit message then re-push here.

@RicGR98
Copy link
Author

RicGR98 commented Jan 9, 2025

Ah OK. That's because you need to link here your upgrade PR: odoo/upgrade#7010

It wasn't authorized in the PR description though ? I'm a bit confused, but I'll put in the commit msg then :)

IAP extracted words are now no longer only linked to
`account.move`'s but can be linked to any model using
`res_model` and `res_id`.

Enterprise: odoo/enterprise#76437
Upgrade: odoo/upgrade#7010
@RicGR98 RicGR98 force-pushed the master-manual-correction-ocr-bank-stmt-rigr branch from 2c8ecb7 to b00e74e Compare January 9, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants