Skip to content

[ADD] migrate database_cleanup - #469

Merged
StefanRijnhart merged 13 commits into
OCA:9.0from
hbrunn:9.0-database_cleanup
Jul 4, 2016
Merged

[ADD] migrate database_cleanup#469
StefanRijnhart merged 13 commits into
OCA:9.0from
hbrunn:9.0-database_cleanup

Conversation

@hbrunn

@hbrunn hbrunn commented Jun 29, 2016

Copy link
Copy Markdown
Member

No description provided.

@hbrunn hbrunn added this to the 9.0 milestone Jun 29, 2016
@@ -1,78 +0,0 @@
# -*- coding: utf-8 -*-

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In order to save the log history of commits by files... Could you use git mv instead of git rm && git add?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank for clarify

Comment thread database_cleanup/README.rst Outdated
Known issues / Roadmap
======================

* ...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unexpected possible title overline or transition.|Treating it as ordinary text because it's so short.
Could we delete this unused section?

@hbrunn

hbrunn commented Jun 29, 2016

Copy link
Copy Markdown
Member Author

@moylop260 thanks, points taken or answered

Comment thread database_cleanup/README.rst Outdated
:alt: Try me on Runbot
:target: https://runbot.odoo-community.org/runbot/149/9.0

For further information, please visit:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remove this part

@moylop260

Copy link
Copy Markdown
Contributor

👍
Thanks for contribution
Excellent work in this refactory!

@luistriana28

Copy link
Copy Markdown

👍
Nice work!

@StefanRijnhart

Copy link
Copy Markdown
Member

👍, thanks!

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.

5 participants