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.

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

[13.0][MIG] project #2280

Merged
merged 2 commits into from
Nov 11, 2020
Merged

[13.0][MIG] project #2280

merged 2 commits into from
Nov 11, 2020

Conversation

Yasaie
Copy link
Member

@Yasaie Yasaie commented Apr 8, 2020

Migration of project

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@Yasaie Yasaie force-pushed the 13.0-mig-project branch from 1386dfe to 9b18de5 Compare April 11, 2020 05:51
@Yasaie Yasaie force-pushed the 13.0-mig-project branch 2 times, most recently from 3072c4d to ecf4f11 Compare April 14, 2020 05:42
@Yasaie Yasaie force-pushed the 13.0-mig-project branch 2 times, most recently from 26b392a to b9b4778 Compare April 15, 2020 08:14
@legalsylvain legalsylvain added this to the 13.0 milestone Apr 27, 2020
@Yasaie Yasaie force-pushed the 13.0-mig-project branch from b9b4778 to e2c0e36 Compare August 9, 2020 07:27
Copy link

@kos94ok-3D kos94ok-3D left a comment

Choose a reason for hiding this comment

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

Multi-company issues

addons/project/migrations/13.0.1.1/post-migration.py Outdated Show resolved Hide resolved
@OCA-git-bot
Copy link
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). 🤖

Yasaie and others added 2 commits November 11, 2020 19:53
* Remove unneeded SQL, as company_id is required on project
* Update module field definition
@pedrobaeza
Copy link
Member

Made some adjustments, so merging now.

@pedrobaeza pedrobaeza merged commit e8abd40 into OCA:13.0 Nov 11, 2020
@kos94ok-3D
Copy link

@pedrobaeza What to do with task without project?

@pedrobaeza
Copy link
Member

Ouch, very good question... restoring through direct commit. That's the problem though for not putting code comments about the why.

@pedrobaeza
Copy link
Member

Restored in 4301baa with proper comment.

@kos94ok-3D
Copy link

Thanks

@pedrobaeza
Copy link
Member

Thanks to you for being vigilant 😃

@Yasaie Yasaie deleted the 13.0-mig-project branch July 20, 2021 05:52
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.

7 participants