-
-
Notifications
You must be signed in to change notification settings - Fork 776
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
[12.0][MIG] report_csv #305
Conversation
3a0ae13
to
f073707
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just some minor comments
f073707
to
356c371
Compare
@etobella Comments attended! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
356c371
to
9268a4c
Compare
@lreficent Fixed! Sorry for the late reply, I didnt see the comment 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works good now. A small comment inline.
Thanks! 👍
9268a4c
to
8dc0ebf
Compare
/ocabot merge |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 516a79e. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
@etobella