Skip to content

[MIG] sale_timesheet_edit: nothing to do#3438

Merged
pedrobaeza merged 1 commit intoOCA:15.0from
daiduongnguyen-odoo:v15_mig_sale_timesheet!
Mar 6, 2023
Merged

[MIG] sale_timesheet_edit: nothing to do#3438
pedrobaeza merged 1 commit intoOCA:15.0from
daiduongnguyen-odoo:v15_mig_sale_timesheet!

Conversation

@daiduongnguyen-odoo
Copy link
Contributor

This PR
Nothing to do

@legalsylvain
Copy link
Contributor

/ocabot migration sale_timesheet_edit

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jul 7, 2022
| |new| sale_timesheet_margin |Nothing to do |No analysis file. No DB layout changes. |
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| sale_timesheet_purchase | | |
| |del| sale_timesheet_purchase |Nothing to do | |
Copy link
Member

Choose a reason for hiding this comment

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

This is not correct. It should be merged with something.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi, sorry i have been out for a longtime and can't quite remember this, do you remember which module it should be merged with?

P/s: probably i will comeback for upgrade 15 -> 16 in june ✌️

Copy link
Member

Choose a reason for hiding this comment

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

No, sorry, but if you don't remember, just remove this change and let the module non handled. It's also a good practice to not handle several modules in the same PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you, i'll keep in mind

@pedrobaeza pedrobaeza merged commit f91fe01 into OCA:15.0 Mar 6, 2023
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