Skip to content
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

Master true notification xdo #32404

Open
wants to merge 40 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@Xavier-Do
Copy link
Contributor

Xavier-Do commented Apr 3, 2019

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the seen 🙂 label Apr 3, 2019

@C3POdoo C3POdoo added the RD label Apr 3, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 4591213 to d30931e Apr 4, 2019

@robodoo robodoo added the CI 🤖 label Apr 4, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from d30931e Apr 4, 2019

@robodoo robodoo removed the CI 🤖 label Apr 4, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch 3 times, most recently Apr 4, 2019

@robodoo robodoo added the CI 🤖 label Apr 4, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch Apr 4, 2019

@robodoo robodoo removed the CI 🤖 label Apr 4, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch 2 times, most recently to 17d02e4 Apr 4, 2019

@robodoo robodoo added the CI 🤖 label Apr 4, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 17d02e4 to d63ec29 Apr 5, 2019

@robodoo robodoo removed the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch 2 times, most recently from b3ed544 to 3e51a1d Apr 5, 2019

@robodoo robodoo added the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 3e51a1d to 1c5af97 Apr 5, 2019

@robodoo robodoo removed the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 1c5af97 to 47ba141 Apr 5, 2019

@robodoo robodoo added the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 47ba141 to 66fb4a9 Apr 5, 2019

@robodoo robodoo removed the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch 2 times, most recently from 51bb2ad to a9d9f52 Apr 5, 2019

@robodoo robodoo added the CI 🤖 label Apr 5, 2019

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from a9d9f52 to 3511b87 Apr 8, 2019

Xavier-Do added some commits Apr 12, 2019

message_post/message_notify new api:
Message post will always be called on a record (ensure_one)
Message_notify can be called on record or on mail_thread, needs at least parent_id or partner_ids/channel_ids to be usefull
Partner_ids parameters should be ids list, not commands

Note: attendee.event_id: not always set, will fail with message_post and useless to call message_notify if no partner_ids
minimize message create queries
Note that some query count may broke if a create is done before since less data are in cache after create.

@Xavier-Do Xavier-Do force-pushed the odoo-dev:master-true-notification-xdo branch from 8f6cd4c to 68b7fdf Apr 17, 2019

@robodoo robodoo added the CI 🤖 label Apr 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.