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

[MIG] account_e-invoice_generate: Migration to 12.0 #83

Merged
merged 7 commits into from
Jan 16, 2019

Conversation

joshuajan
Copy link
Contributor

No description provided.

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jan 10, 2019
@pedrobaeza
Copy link
Member

Can you please see Travis problem and squash your migration commits into one (as they are not very meaningful)?

@OCA-git-bot OCA-git-bot mentioned this pull request Jan 10, 2019
18 tasks
@astirpe
Copy link
Member

astirpe commented Jan 11, 2019

Please rebase to fix the travis

@joshuajan joshuajan force-pushed the 12.0-mig-account_e-invoice_generate branch from 8628328 to 68c7ae4 Compare January 13, 2019 16:30
@joshuajan joshuajan changed the title 12.0 mig account e invoice generate [MIG] account_e-invoice_generate: Migration to 12.0 Jan 14, 2019
@joshuajan
Copy link
Contributor Author

Dear all, I have rebase and make all change into one commit.

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.

Thanks!

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.

It would be great that you use the README by fragments system: https://github.com/OCA/maintainer-tools/#readme-generator

account_e-invoice_generate/__manifest__.py Outdated Show resolved Hide resolved
@pedrobaeza
Copy link
Member

In fact, README by fragment is already used. You have to update that fragments instead for contributors part, and run the utility (or let the bot to run it after the merge, but at least the first step).

@joshuajan joshuajan force-pushed the 12.0-mig-account_e-invoice_generate branch from 68c7ae4 to 0412841 Compare January 14, 2019 15:43
@joshuajan
Copy link
Contributor Author

Dear @pedrobaeza Thanks for your reply. I have update fragments.

@pedrobaeza pedrobaeza merged commit 2f34d81 into OCA:12.0 Jan 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants