Skip to content

[FIX] migrate categories of hr.holiday.status - #350

Merged
StefanRijnhart merged 3 commits into
OCA:7.0from
hbrunn:7.0-hr_holidays-migrate_categ_id
Aug 19, 2015
Merged

[FIX] migrate categories of hr.holiday.status#350
StefanRijnhart merged 3 commits into
OCA:7.0from
hbrunn:7.0-hr_holidays-migrate_categ_id

Conversation

@hbrunn

@hbrunn hbrunn commented Aug 17, 2015

Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this not be called openupgrade.get_legacy_name('crm_case_categ_id')?

@pedrobaeza

Copy link
Copy Markdown
Member

Please remember also to add a PR to modify migration status on docs of the 8.0 branch.

@pedrobaeza

Copy link
Copy Markdown
Member

OK, I see it's already marked as done, so forget it.

@hbrunn

hbrunn commented Aug 18, 2015

Copy link
Copy Markdown
Member Author

@StefanRijnhart done

@pedrobaeza

Copy link
Copy Markdown
Member

I have checked that crm_case_categ are not inserted twice, so everything seems OK

👍

@StefanRijnhart

Copy link
Copy Markdown
Member

👍

StefanRijnhart pushed a commit that referenced this pull request Aug 19, 2015
[FIX] migrate categories of hr.holiday.status
@StefanRijnhart
StefanRijnhart merged commit 9327fe3 into OCA:7.0 Aug 19, 2015
NL66278 pushed a commit to NL66278/OpenUpgrade that referenced this pull request Apr 15, 2016
…eate

[FIX] use the column._symbol-c as placeholder for the column value

When generating the SQL query in the low level '_create' method from
BaseModel, column._symbol-c must be used as placeholder as for the
method 'set' of the column itself. Otherwise it's no more possible to define specialized column.
GuillemCForgeFlow pushed a commit to ForgeFlow/OpenUpgrade that referenced this pull request Oct 10, 2024
[ADD] openupgrade_160: fill_analytic_distribution
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.

3 participants