Skip to content

[MIG] Migrate database_cleanup on 9.0 - #415

Closed
adrienpeiffer wants to merge 2 commits into
OCA:9.0from
acsone:9.0-port-database-cleanup-ape
Closed

[MIG] Migrate database_cleanup on 9.0#415
adrienpeiffer wants to merge 2 commits into
OCA:9.0from
acsone:9.0-port-database-cleanup-ape

Conversation

@adrienpeiffer

Copy link
Copy Markdown

No description provided.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-12.08%) to 52.189% when pulling a68e0db on acsone:9.0-port-database-cleanup-ape into 77f9ce5 on OCA:9.0.

@pedrobaeza pedrobaeza mentioned this pull request Apr 23, 2016
59 tasks
Comment thread database_cleanup/__openerp__.py Outdated
{
'name': 'Database cleanup',
'version': '8.0.0.1.0',
'version': '9.0.0.1.0',

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.

9.0.1.0.0

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-12.08%) to 52.189% when pulling e004b7e on acsone:9.0-port-database-cleanup-ape into 77f9ce5 on OCA:9.0.

@tremlin

tremlin commented Apr 27, 2016

Copy link
Copy Markdown

Code review okay, brief test in runbot 👍

@astirpe

astirpe commented Jun 10, 2016

Copy link
Copy Markdown
Member

Tested on v9, it works properly 👍 Code review also ok

@hbrunn

hbrunn commented Jun 29, 2016

Copy link
Copy Markdown
Member

@adrienpeiffer I migrated the code to the new api and did some improvements (like tests) in #469, do you agree to close this one?

@moylop260

moylop260 commented Jun 29, 2016

Copy link
Copy Markdown
Contributor

@hbrunn 👍 use a PR with new api changes too #469

@adrienpeiffer

Copy link
Copy Markdown
Author

@hbrunn ok, no problem

@moylop260 moylop260 closed this Jun 29, 2016
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (10.0)
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.

7 participants