Conversation
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
CRogos
left a comment
There was a problem hiding this comment.
Maybe you should rebase this and remove the test-requirments.txt
dependency moved repo: https://github.com/OCA/mail/tree/18.0/mail_activity_board
You could also squash some of the commits: https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests
@glitchov are you going on with these comments? |
|
@CRogos , @sergiocorato : Sorry i missed CRogos' review so i'm gonna take a look as soon as i have time for it. |
[IMP] Change partner_id to a computed field [IMP] Add tests to increase coverage
Currently translated at 100.0% (3 of 3 strings) Translation: social-16.0/social-16.0-mail_activity_partner Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_partner/it/
Currently translated at 100.0% (3 of 3 strings) Translation: social-16.0/social-16.0-mail_activity_partner Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_partner/es/
Currently translated at 100.0% (3 of 3 strings) Translation: social-16.0/social-16.0-mail_activity_partner Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_partner/es/
22fc0f8 to
d685a15
Compare
|
New PR: OCA/mail#175 |
Migration from 16.0