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

[10.0] l10n_lu_mis_reports_tax #27

Merged
merged 10 commits into from Dec 5, 2017

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Nov 2, 2017

No description provided.

First commit, restoring the tax tags that were dropped in Odoo's l10n_lu
in PR odoo/odoo#19102.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 94a5ce6 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 94a5ce6 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 94a5ce6 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 94a5ce6 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@sbidoul sbidoul force-pushed the 10.0-l10n_lu_mis_reports_tax-sbi branch from 94a5ce6 to 630095f Compare November 2, 2017 13:21
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2c3187b on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2c3187b on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@coveralls
Copy link

coveralls commented Nov 2, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 1b9f110 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@sbidoul sbidoul force-pushed the 10.0-l10n_lu_mis_reports_tax-sbi branch from 1b9f110 to 5e0f7d7 Compare November 3, 2017 09:08
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5e0f7d7 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5e0f7d7 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@coveralls
Copy link

coveralls commented Nov 3, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling e92bcf4 on acsone:10.0-l10n_lu_mis_reports_tax-sbi into 65d189c on OCA:10.0.

@sbidoul
Copy link
Member Author

sbidoul commented Nov 4, 2017

Runbot is red (orange actually) due to warnings about l10n_lu.* tags being defined in this module instead of l10n_lu.

We need to declare the tags l10n_lu.* to avoid duplicating them in case they have been preserved thanks to odoo/odoo@c44024b#diff-11b9f8c1c0e3af4ea1a555a7c31bc883.

Suggestions welcome.

@adrienpeiffer
Copy link

Issue related to the init hook
odoo/odoo#20792

Copy link

@adrienpeiffer adrienpeiffer left a comment

Choose a reason for hiding this comment

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

👍

@sbidoul sbidoul merged commit ddd1e8f into OCA:10.0 Dec 5, 2017
@sbidoul sbidoul deleted the 10.0-l10n_lu_mis_reports_tax-sbi branch December 5, 2017 12:35
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

3 participants