Skip to content

Conversation

@u0f
Copy link

@u0f u0f commented Nov 17, 2025

Migration to version 18:

  • Update README
  • Give support all document layouts
  • Delete the data file to prevent the malpractice of creating a system parameter throught it.

https://www.loom.com/share/4a51c8b870d24a23be625cb98d346b38

MT-11818 @moduon @Shide @chienandalu

@u0f u0f force-pushed the 18.0-mig-report_display_name_in_footer branch 2 times, most recently from ae6a5f1 to d4c5c5c Compare November 17, 2025 10:53
@u0f u0f changed the title [MIG] report_display_name_in_footer: Migration to 18.0 [18.0][MIG] report_display_name_in_footer: Migration to 18.0 Nov 17, 2025
@u0f u0f changed the title [18.0][MIG] report_display_name_in_footer: Migration to 18.0 [18.0][MIG] report_display_name_in_footer Nov 17, 2025
@u0f u0f changed the title [18.0][MIG] report_display_name_in_footer [18.0][MIG] report_display_name_in_footer: Migration to 18.0 Nov 17, 2025
@u0f u0f force-pushed the 18.0-mig-report_display_name_in_footer branch from d4c5c5c to 4d5bf46 Compare November 17, 2025 11:04
Copy link

@Shide Shide left a comment

Choose a reason for hiding this comment

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

Great migration! Congrats!
I love the Implementation section, thanks ❤️

Only a word change.

@u0f u0f force-pushed the 18.0-mig-report_display_name_in_footer branch from 4d5bf46 to c880503 Compare November 17, 2025 14:05
@u0f u0f force-pushed the 18.0-mig-report_display_name_in_footer branch from c880503 to e5a2bfd Compare November 17, 2025 14:14
Copy link

@fcvalgar fcvalgar left a comment

Choose a reason for hiding this comment

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

Functional review,

All available report layouts in Odoo 18 have been fully tested.

Examples:

image image image

Thank you very much for this work, and congratulations to @u0f for the fantastic contribution.

@u0f u0f requested a review from Shide November 24, 2025 07:43
@u0f
Copy link
Author

u0f commented Nov 24, 2025

Thanks for the reviews and the comments! Can this be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants