Skip to content

[17.0][MIG] report_py3o: Migration to version 17.0 #1021

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 115 commits into from
May 22, 2025
Merged

[17.0][MIG] report_py3o: Migration to version 17.0 #1021

merged 115 commits into from
May 22, 2025

Conversation

pilarvargas-tecnativa
Copy link

@Tecnativa TT51940

superseder #931

The migration commits to 17 of the RP that had been lost in the last rebasing have been recovered respecting the authorship of the RP being replaced.

@victoralmau @carlos-lopez-tecnativa please review

lmignon and others added 30 commits May 15, 2025 13:03
@lmignon @pilarvargas-tecnativa
Move to report_py3o after import from HG
60af0f6
@lmignon @pilarvargas-tecnativa
Add setup
e491cb3
@lmignon @pilarvargas-tecnativa
[FIX] View inheritance may not use attribute 'string' as a selector.
3d25112
@lmignon @pilarvargas-tecnativa
[IMP] New API + Short Hearder
07b41d3
@lmignon @pilarvargas-tecnativa
Remove unicode string
7f0bcb3
@lmignon @pilarvargas-tecnativa
[IMP] Add the possiblity to get the template from an absolute path on…
e92597c 
… the server
@lmignon @pilarvargas-tecnativa
[IMP] Allow user to use local fusion
306081c
@lmignon @pilarvargas-tecnativa
Add a simple way to extend the parser context
bc8b245
@lmignon @pilarvargas-tecnativa
Remove ACL on removed model
d184a15
@lmignon @pilarvargas-tecnativa
Improve filetype selection name
dc991d2
@lmignon @pilarvargas-tecnativa
Fix native flag
9454c99
@faide @pilarvargas-tecnativa
fixed missing comma in model definition
c87ecf4
@lmignon @pilarvargas-tecnativa
Add simple tests and fix call to fusion server
5990931
@lmignon @pilarvargas-tecnativa
Fix skip fusion on server when the fusion is local
72d4897
@lmignon @pilarvargas-tecnativa
Improve tests
9dba42e
@lmignon @pilarvargas-tecnativa
Replace openerp element by odoo
b0162c8
@alexis-via @pilarvargas-tecnativa
Small usability improvements
ccc8d46 
Replace README.md by README.rst (not finished)
@alexis-via @pilarvargas-tecnativa
Take into accounts most remarks of @lasley
9bb0021 
Remove <data> in views
Protect import of py3o libs
Remove dep on base module
Other small changes
@lmignon @pilarvargas-tecnativa
Replace plain SQL by orm
881a1a9
@lmignon @pilarvargas-tecnativa
rename module ir_report to ir_actions_report_xml
5df0331
@lmignon @pilarvargas-tecnativa
Add more tests and fixes issues found by tests
ab50643
@lmignon @pilarvargas-tecnativa
[FIX] Check constrains only if report_type == 'py3o'
52fc747
@alexis-via @pilarvargas-tecnativa
report_py3o: full installation procedure in README
e0cf82a
@alexis-via @pilarvargas-tecnativa
Port report_py3o to Odoo v10
797472c
@alexis-via @pilarvargas-tecnativa
Update version numbers
bad036e 
Rename __openerp__.py to __manifest__.py
@alexis-via @pilarvargas-tecnativa
PEP8 fix
4439276
@alexis-via @pilarvargas-tecnativa
Update README
f68b3ab
@alexis-via @pilarvargas-tecnativa
FIX double dot in filename
d1ef5e1
@alexis-via @pilarvargas-tecnativa
Add access to report_xml object in report
a24e3da
@alexis-via @pilarvargas-tecnativa
FIX Adapt test suite to my recent commit to avoid double dot on filen…
4343916 
…ame extension
mymage and others added 11 commits May 15, 2025 18:06
@mymage @pilarvargas-tecnativa
Translated using Weblate (Italian)
9f73ad7 
Currently translated at 100.0% (46 of 46 strings)

Translation: reporting-engine-16.0/reporting-engine-16.0-report_py3o
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_py3o/it/
@CRogos @pilarvargas-tecnativa
[IMP] remove python libs included in odoo from requirements
6dbab87
@jakobkrabbe @pilarvargas-tecnativa
Translated using Weblate (Swedish)
6a81141 
Currently translated at 100.0% (46 of 46 strings)

Translation: reporting-engine-16.0/reporting-engine-16.0-report_py3o
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_py3o/sv/
@jdoutreloux @pilarvargas-tecnativa
[FIX] _py3o_parser_context: fix call to format_date in old_format_lang
6269582
@imbarbudiman @pilarvargas-tecnativa
[FIX] report_py3o: change display_address() to _display_address()
bfbc52f
@ThomasBinsfeld @pilarvargas-tecnativa
[IMP] report_py3o: use the standard file name behavior when zipping r…
e197fba 
…eport files
@mymage @pilarvargas-tecnativa
Translated using Weblate (Italian)
fc17c9c 
Currently translated at 100.0% (46 of 46 strings)

Translation: reporting-engine-16.0/reporting-engine-16.0-report_py3o
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-report_py3o/it/
@kluna1998 @pilarvargas-tecnativa
[IMP] report_py3o: refresh
5b887e3
@bug2point0 @pilarvargas-tecnativa
[IMP] report_py3o: pre-commit auto fixes
3d113a9
@bug2point0 @pilarvargas-tecnativa
[17.0] [MIG] report_py3o
61b46f7
@bug2point0 @pilarvargas-tecnativa
[IMP] replace deprecated pkg_resources with importlib.resources
a066f5b
@pilarvargas-tecnativa pilarvargas-tecnativa mentioned this pull request May 15, 2025
Closed
@pedrobaeza
Copy link
Member

/ocabot migration report_py3o

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone May 15, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request May 15, 2025
Open
26 tasks
carlos-lopez-tecnativa
carlos-lopez-tecnativa approved these changes May 16, 2025
View reviewed changes
Copy link

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Non-blocking comment: This section of the README is a bit confusing. I think it needs to be formatted in a more readable way, such as clearly differentiating the variable names from their descriptions. What do you think?

image

@pilarvargas-tecnativa
Copy link
Author

LGTM Non-blocking comment: This section of the README is a bit confusing. I think it needs to be formatted in a more readable way, such as clearly differentiating the variable names from their descriptions. What do you think?

Done, thanks!

victoralmau
victoralmau approved these changes May 22, 2025
View reviewed changes
Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review OK

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-1021-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 86f4e87 into OCA:17.0 May 22, 2025
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-mig-report_py3o branch May 22, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victoralmau victoralmau victoralmau approved these changes

@carlos-lopez-tecnativa carlos-lopez-tecnativa carlos-lopez-tecnativa approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
17.0
Development

Successfully merging this pull request may close these issues.