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

Fix errors on project_issue migration #87

Closed
wants to merge 1 commit into from

Conversation

dreispt
Copy link
Sponsor Member

@dreispt dreispt commented Jul 21, 2014

Fixed errors resulting in exceptions when trying to upgrade my database.

Notably, I can't find the openupgrade.map_values() function anywhere in the source code. Maybe it was missing from some commit?

@StefanRijnhart
Copy link
Member

map_values is added in #80, which was reverted but has now been waiting for approval for a long time. I think that means you can uncomment the lines that deal with the priority field, right?

@@ -21,7 +21,7 @@

from openerp.openupgrade import openupgrade

column_renames = {'project_issue': ['priority']}
#column_renames = {'project_issue': ['priority']}

Copy link
Member

Choose a reason for hiding this comment

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

Should be {'project_issue': [('priority', None)]}. Sorry for my mistakes!

@legalsylvain legalsylvain added this to the 8.0 milestone Oct 21, 2014
@hbrunn
Copy link
Member

hbrunn commented Nov 10, 2014

this is done in #128 now

@hbrunn hbrunn closed this Nov 10, 2014
tranngocson1996 pushed a commit to tranngocson1996/OpenUpgrade that referenced this pull request Jan 25, 2022
Merge from upstream 14 220122 01
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.

None yet

4 participants