Skip to content
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

[16.0][MIG] mail_tracking: Migration to 16.0 #1191

Closed
wants to merge 1 commit into from
Closed

[16.0][MIG] mail_tracking: Migration to 16.0 #1191

wants to merge 1 commit into from

Conversation

payen000
Copy link
Contributor

No description provided.

@payen000 payen000 force-pushed the 16.0-mail_tracking-mig-payen branch from 3dd5e61 to 33f3680 Compare July 27, 2023 23:59
@payen000 payen000 force-pushed the 16.0-mail_tracking-mig-payen branch from 33f3680 to 72e722b Compare July 28, 2023 00:11
@rvalyi
Copy link
Member

rvalyi commented Jul 28, 2023

hello @payen000 thank you for the work, but please you should follow the migration guidelines here https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-16.0 Could you re-do the PR properly please?

@payen000
Copy link
Contributor Author

Hello @rvalyi,

Thank you for your comment, I'll be checking the guidelines and applying the relevant changes soon 👍

@payen000 payen000 changed the title [MIG] mail_tracking: migration to 16.0 [16.0][MIG] mail_tracking: Migration to 16.0 Jul 28, 2023
@rvalyi
Copy link
Member

rvalyi commented Jul 28, 2023

Hello @rvalyi,

Thank you for your comment, I'll be checking the guidelines and applying the relevant changes soon +1

You may also want to try with this tool https://github.com/OCA/odoo-module-migrator It does enforce the guidelines and makes everything easier.

@bhaveshselarka bhaveshselarka mentioned this pull request Aug 1, 2023
Closed
@astirpe
Copy link
Member

astirpe commented Aug 1, 2023

There is already #1091

@payen000
Copy link
Contributor Author

payen000 commented Aug 1, 2023

There is already #1091

Thank you @astirpe, I will contribute to that PR instead.

@payen000
Copy link
Contributor Author

payen000 commented Aug 1, 2023

Hello @rvalyi,
Thank you for your comment, I'll be checking the guidelines and applying the relevant changes soon +1

You may also want to try with this tool https://github.com/OCA/odoo-module-migrator It does enforce the guidelines and makes everything easier.

Thank you for the help @rvalyi, I was not aware of the existing migration for this module when I created this PR, so I will be contributing there instead.

@payen000 payen000 closed this Aug 1, 2023
@payen000 payen000 mentioned this pull request Sep 5, 2023
Merged
payen000 added a commit to vauxoo-dev/social that referenced this pull request Sep 21, 2023
@payen000 @filoquin
[MIG] mail_tracking: migration to 16.0
77df5b8 
Redo of PRs

- OCA#1091
- OCA#1191

Co-authored-by: filoquin <filoquin@gmail.com>
payen000 added a commit to vauxoo-dev/social that referenced this pull request Sep 21, 2023
@payen000 @filoquin
[MIG] mail_tracking: migration to 16.0
eae4b50 
Redo of PRs

- OCA#1091
- OCA#1191

Co-authored-by: filoquin <filoquin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@payen000 @rvalyi @astirpe