[11.0][IMP] crm_claim: add activities to claims #214
Conversation
oihane
changed the title
| activity_type = env.ref('mail.mail_activity_data_todo') | ||
| crm_claim_model = env.ref('crm_claim.model_crm_claim') | ||
| cr.execute(""" | ||
| SELECT |
There was a problem hiding this comment.
You can make the insert from a select query and make this faster
| WHERE | ||
| date_action_next IS NOT Null | ||
| OR | ||
| action_next IS NOT Null; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| id, | ||
| name, | ||
| user_id, | ||
| CASE WHEN date_action_next IS NOT Null THEN date_action_next |
There was a problem hiding this comment.
Use COALESCE(date_action_next, now()) instead for simplifying the query
| FROM | ||
| crm_claim | ||
| WHERE | ||
| date_action_next IS NOT Null |
There was a problem hiding this comment.
I'm thinking that you can remove now this part of the WHERE
There was a problem hiding this comment.
Before this, you can set a 'date_action_next' without adding an 'action_next', if you don't add this where it won't create any activity (a 'summary' is not mandatory in the database, you can create one without text)
There was a problem hiding this comment.
Is the text mandatory on activities?
There was a problem hiding this comment.
There are 3 fields mandatory through odoo:
- user_id
- date_deadline
- activity_type_id
Internally:
- res_model
- res_model_id
- res_id
- res_name
summary and note are NOT MANDATORY
There was a problem hiding this comment.
OK, thanks for the confirmation.
No description provided.