Skip to content

[10.0][MIG] account_voucher - #845

Merged
hbrunn merged 2 commits into
OCA:10.0from
ForgeFlow:10.0-account-voucher
Jun 6, 2017
Merged

[10.0][MIG] account_voucher#845
hbrunn merged 2 commits into
OCA:10.0from
ForgeFlow:10.0-account-voucher

Conversation

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor

This PR adds the migration scripts to v10 for account_voucher module.

@LoisRForgeFlow LoisRForgeFlow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍

Good Job!

@AaronHForgeFlow AaronHForgeFlow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done 🥇

@cubells cubells left a comment

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.

A typo

@cubells cubells May 29, 2017

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.

Typo: there are 4 extra spaces... :)

@cubells cubells left a comment

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.

👍

LGTM

""")


@openupgrade.migrate(use_env=True)

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.

use_env=True is default so it's not necessary but this not block pr.

@NL66278 NL66278 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This contains invalid SQL syntax.

Tip: to check SQL statements, just past them in the psql console (connected to a database migrated to 10.0).

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor Author

@NL66278 comment attended to, thanks!! My mind thinks in MySQL instead of PostgreSQL 😅

@hbrunn
hbrunn merged commit 647bff1 into OCA:10.0 Jun 6, 2017
@MiquelRForgeFlow
MiquelRForgeFlow deleted the 10.0-account-voucher branch June 14, 2017 09:13
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