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] l10n_us_gaap_mis_report #54

Merged
merged 7 commits into from
Mar 23, 2020

Conversation

JoanSForgeFlow
Copy link
Contributor

l10n_us_gaap_mis_report migration to v13.
Depends on #53 and https://github.com/OCA/mis-builder/tree/13.0-staging/mis_builder

cc ~ @ForgeFlow @LoisRForgeFlow

@OCA-git-bot
Copy link
Contributor

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

@AdriaGForgeFlow AdriaGForgeFlow force-pushed the 13.0-mig-l10n_us_gaap_mis_report branch 2 times, most recently from 4103244 to 96328cd Compare March 3, 2020 10:41
@AdriaGForgeFlow AdriaGForgeFlow force-pushed the 13.0-mig-l10n_us_gaap_mis_report branch 2 times, most recently from 3f0086a to 56922d8 Compare March 23, 2020 07:33
@AdriaGForgeFlow AdriaGForgeFlow force-pushed the 13.0-mig-l10n_us_gaap_mis_report branch from 56922d8 to 9e0c87c Compare March 23, 2020 07:39
Copy link

@LoisRForgeFlow LoisRForgeFlow 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, LGTM

@JordiBForgeFlow
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-54-by-JordiBForgeFlow-bump-no, awaiting test results.

Copy link

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

👍

@OCA-git-bot OCA-git-bot merged commit d5c58b6 into OCA:13.0 Mar 23, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 2c9fd11. 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.

8 participants