[16.0][OU-ADD] project: migration script #4289
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
remytms
force-pushed
the
16-add-project
branch
from
4788003 to
a5a03da
Compare
remytms
changed the title
remytms
force-pushed
the
16-add-project
branch
2 times, most recently
from
c358606 to
b785e86
Compare
|
I don't remember if I should modify the file |
|
Yes if something shouldn't be imported, but commenting the lines. |
|
/ocabot migration project |
pedrobaeza
requested changes
Feb 13, 2024
openupgrade_scripts/scripts/project/16.0.1.2/upgrade_analysis_work.txt
Outdated
Show resolved
Hide resolved
|
|
||
| project / project.task / is_blocked (boolean) : NEW isfunction: function, stored | ||
| project / project.task / milestone_id (many2one) : NEW relation: project.milestone, hasdefault: compute | ||
| # TODO (speed improvement): pre-migration: pre-compute value for new computed fields |
There was a problem hiding this comment.
No, I'm not going to do this. I focus on getting something that works.
This was referenced Feb 13, 2024
|
I will work on this tomorrow. :) |
remytms
force-pushed
the
16-add-project
branch
from
b785e86 to
26daeaf
Compare
remytms
commented
Feb 15, 2024
Comment on lines
+20
to
+23
| pad_project / project.project / description_pad (char) : DEL | ||
| pad_project / project.project / use_pads (boolean) : DEL | ||
| pad_project / project.task / description_pad (char) : DEL | ||
| # TODO: Module pad_project has been removed |
There was a problem hiding this comment.
@pedrobaeza I'm not sure what to do with this. Should I rename the table with a legacy name to ensure that data are not deleted ? Or should I just do nothing ?
There was a problem hiding this comment.
I would say "Nothing to do", as the content is dumped on save to the description fields, and the collaboration comes now from web editor itself.
remytms
force-pushed
the
16-add-project
branch
from
26daeaf to
32ad4a2
Compare
robinkeunen
suggested changes
Feb 15, 2024
Linekio
reviewed
Feb 16, 2024
remytms
force-pushed
the
16-add-project
branch
2 times, most recently
from
007f10e to
c21d9f8
Compare
remytms
force-pushed
the
16-add-project
branch
from
c21d9f8 to
a86c42d
Compare
pedrobaeza
approved these changes
Mar 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cortesy @SonCrits
Replace #3989
Speed improvements can be done by computing some new computed fields via SQL request. But I will not do that in this PR. As it is not mandatory.