Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[13.0][MIG] account_bank_statement_import_move_line: Migration to 13.0 #354

Conversation

joao-p-marques
Copy link
Member

@joao-p-marques joao-p-marques commented Nov 2, 2020

Superseeds #241, as it seems like a stopped WIP.

  • Rebase from current commits in 13.0 branch.
  • Squash administrative commits
  • Carry on with migration (invoice_id -> move_id in particular)

TT26517

ping @pedrobaeza @ernestotejeda

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@joao-p-marques joao-p-marques mentioned this pull request Nov 2, 2020
14 tasks
Co-authored-by: Pedro Gonzalez <pegonzalezs@gmail.com>
Co-authored-by: Joao Marques <joao.marques@tecnativa.com>
@joao-p-marques joao-p-marques force-pushed the 13.0-mig-account_bank_statement_import_move_line branch from 1e3b616 to fdc1ade Compare November 2, 2020 12:38
Copy link
Member

@ernestotejeda ernestotejeda left a comment

Choose a reason for hiding this comment

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

Code review 👍

@pedrobaeza pedrobaeza added this to the 13.0 milestone Nov 5, 2020
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-354-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 29e9c41 into OCA:13.0 Nov 5, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9014bfc. Thanks a lot for contributing to OCA. ❤️

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.

None yet

8 participants