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] crm_phonecall_summary_predefined #252
[12.0] [MIG] crm_phonecall_summary_predefined #252
Conversation
Please squash together adjacent "[UPD] Update crm_phonecall_summary_predefined.pot" commits and check runbot status. |
906b30d
to
b6898b7
Compare
Done
Well regarding that, took a pull from the remote 12.0 branch which has |
Fixed |
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.
@anandkansagra thanks for your work.
I did a code review and here are some minor changes:
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
crm_phonecall_summary_predefined/views/crm_phonecall_summary_view.xml
Outdated
Show resolved
Hide resolved
@tbaden , |
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.
Code Review: LGTM
This module fills the gap of phone marketing companies where phone call summaries can only belong to a limited amount of possibilities.
0785735
to
bd3a5ec
Compare
Issue Reference