[IMP] util/fields: dump_field_to_chatter column fallback#455
Conversation
|
|
| ON batch.id = t.id | ||
| {fk_join} | ||
| {value_join} | ||
| GROUP BY t.{res_id} |
There was a problem hiding this comment.
Does it work with the "explode" done on t.id?
There was a problem hiding this comment.
It looks like it doesn't. I think we need to fix the batch CTE below. I will check.
There was a problem hiding this comment.
I pushed a different version. Now with clear different queries. There is a test as well for parallel updates.
ed55699 to
ac3a4c2
Compare
| (``jsonb``) columns are written through their ``en_US`` | ||
| key. | ||
| :param str where: extra SQL ``WHERE`` condition on the source records. It is a | ||
| ``str.format`` template whose ``{0}`` is the source table alias; use it |
There was a problem hiding this comment.
This is a slight change, since this util was introduced very recently breaking scripts should be minimal. I could introduce a static check assert where is None or '{0}' in where to avoid missing it.
There was a problem hiding this comment.
It may work without the table alias if there are no ambiguities.
ac3a4c2 to
d5de3ee
Compare
|
upgradeci retry with always only crm hr event |
For the cases when `mail` is not installed.
e2d8e35 to
f4ae0e3
Compare
|
I've rebased to solve conflicts. This should be good to go. |
|
@aj-fuentes @KangOl linked pull request(s) odoo/upgrade#10571 not ready. Linked PRs are not staged until all of them are ready. |
For the cases when `mail` is not installed. closes #455 Related: odoo/upgrade#10571 Signed-off-by: Christophe Simonis (chs) <chs@odoo.com>
For the cases when `mail` is not installed. closes #455 Related: odoo/upgrade#10571 Signed-off-by: Christophe Simonis (chs) <chs@odoo.com>
3 participants
For the cases when
mailis not installed.