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

[9.0][MIG] invoice_fiscal_position_update: Migration to 9.0 #223

Merged

Conversation

sergio-teruel
Copy link
Contributor

@sergio-teruel sergio-teruel commented Jan 31, 2017

  • More tests
  • Minor fixes and improvements.

cc @Tecnativa
@pedrobaeza Can we close this PR: #196 ??

@pedrobaeza pedrobaeza changed the title [9.0][IMP] invoice_fiscal_position_update: More tests for increase coverall and minor fixes [9.0][MIG] account_invoice_fiscal_position_update: Migration to 9.0 Jan 31, 2017
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Please squash together Angel's commits

@sergio-teruel sergio-teruel force-pushed the 9.0-MIG-tec-invoice_fiscal_position_update branch from 8a634c8 to 28dd7c1 Compare January 31, 2017 14:58
@sergio-teruel
Copy link
Contributor Author

@pedrobaeza squash done.

@sergio-teruel sergio-teruel changed the title [9.0][MIG] account_invoice_fiscal_position_update: Migration to 9.0 [9.0][MIG] invoice_fiscal_position_update: Migration to 9.0 Jan 31, 2017
Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Thanks

@pedrobaeza pedrobaeza merged commit 22553ac into OCA:9.0 Jan 31, 2017
@pedrobaeza pedrobaeza deleted the 9.0-MIG-tec-invoice_fiscal_position_update branch January 31, 2017 16:53
BT-dherreros pushed a commit to BT-dherreros/account-invoicing that referenced this pull request Apr 29, 2019
Syncing from upstream OCA/account-invoicing (12.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.

5 participants