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

[14.0][MIG] account_payment_unece #100

Closed
wants to merge 17 commits into from

Conversation

alexis-via
Copy link
Contributor

No description provided.

@alexis-via
Copy link
Contributor Author

Requires PR #98 and OCA/bank-payment#744

@pedrobaeza pedrobaeza mentioned this pull request Oct 17, 2020
8 tasks
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.

Similar comments as account_tax_unece

@pedrobaeza
Copy link
Member

You can take the occasion to rebase when the main module is merged.

@alexis-via alexis-via force-pushed the 14-mig-account_payment_unece branch from 7abd678 to 187eeb7 Compare October 17, 2020 21:24
Copy link
Member

@astirpe astirpe left a comment

Choose a reason for hiding this comment

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

Same comment as #99 (comment)

@phuctranfxvn
Copy link

Hello, I think you should include setup/account_payment_unece in your PR. This will be required for other migration.
Thanks

@astirpe
Copy link
Member

astirpe commented Jan 8, 2021

Just squash the last two commits and this PR can be merged IMO

@rconjour
Copy link

I agree with @astirpe , please squas the last two commits, and we can continue with merging :).

@manengel
Copy link

Hello @alexis-via ,
this Pull Request has been open since a while and there was quite alot of activity.
Do you plan to continue / finish the work here? It would be great to know, thank you in advance.

@rconjour
Copy link

rconjour commented Apr 8, 2021

@alexis-via @manengel created superseding PR #107 so we can get this merged, and continue on the edi repo.

@StefanRijnhart
Copy link
Member

Continued on #107

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.