Skip to content

[19.0] [MIG] purchase_report_date_format: Migration to 19.0#92

Open
BhaveshHeliconia wants to merge 10 commits into
OCA:19.0from
HeliconiaIO:19.0-mig-purchase_report_date_format
Open

[19.0] [MIG] purchase_report_date_format: Migration to 19.0#92
BhaveshHeliconia wants to merge 10 commits into
OCA:19.0from
HeliconiaIO:19.0-mig-purchase_report_date_format

Conversation

@BhaveshHeliconia

Copy link
Copy Markdown
Contributor

No description provided.

@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Dec 26, 2025
5 tasks

@WesleyOliveira98 WesleyOliveira98 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@marcelsavegnago marcelsavegnago left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@BhaveshHeliconia

Copy link
Copy Markdown
Contributor Author

@HviorForgeFlow Could you please review and merge this?

Comment on lines +9 to +13
Purchase Order print:

- Order Date
- Order Deadline
- Requested Date

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This part was removed in the previous migration to version 18.0

I would like to know why you took that decision, at the time to trigger the merge for version 18.0 I didn't realized about this change

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

image I don't remember making that change during the 18.0 migration. Could you point me to the commit where purchase_report_date_format/readme/DESCRIPTION.md was removed?

@HviorForgeFlow HviorForgeFlow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants