SirPyTech

Results 26 comments of SirPyTech

> My comment is still pending [#441 (comment)](https://github.com/OCA/dms/pull/441#issuecomment-3196129887) > > _I think all these changes can be combined into a single commit: [IMP] dms_field: Propagate groups from field template_ >...

> /ocabot merge nobump (from https://github.com/OCA/account-payment/pull/859#issuecomment-3592697369) @rvalyi please try again

Thanks for the quick and useful feedback 🤗, I'll try to create an anonymized PDF that reproduces this issue. Do you have a link to the newer PDF specification? I...

An update: the `PyPDF2==3.0.1` can read the PDF correctly. Still working on an anonymized version of the file to add to this issue.

@stefan6419846 I included the PDF 🚀 please remove the label https://github.com/py-pdf/pypdf/issues?q=label%3Aneeds-pdf

Thanks for analyzing this issue so carefully! > In case you are in control of the generator, you might want to check if this can be fixed there as well....

@simahawk here I addressed your latest comments https://github.com/akretion/rest-framework/pull/8#discussion_r2204278496 and https://github.com/akretion/rest-framework/pull/8#issuecomment-3068521632, please have a look

The error in the tests (https://github.com/OCA/rest-framework/actions/runs/17124124402/job/48571725570?pr=554#step:8:120) does not seem to be related to these changes Stack 2025-08-21 10:24:04,335 340 INFO odoo odoo.modules.loading: Loading module graphql_base (12/53) 2025-08-21 10:24:04,475 340 CRITICAL...

Rebased to check if this is affected by https://github.com/fastapi/fastapi/commit/51ad909ffe9f5b2d5c9315554e75e39a8a2d725c. It still looks good :sunglasses: