Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 28, 2025

Before this commit, customers were facing problems during the first synchronization related to the abundance of invitations sent through Outlook from previously created events in Odoo side. This should not happen because most of the time it is not useful synchronizing events that were already created before starting the synchronization with Outlook (from feedbacks discussed with customers and internally).

After this commit, we no longer synchronize events that were created before the first synchronization of any user in a database if we don't find any token created in it (i.e. if no user synchronized its Odoo Calendar with Outlook before this improvement). In case of any user already have synchronized its calendar with Outlook, we won't change the synchronization behavior (since the biggest part of the invitations were already sent).

task-4294884

Forward-Port-Of: #200957
Forward-Port-Of: #185911

@robodoo
Copy link
Contributor

robodoo commented Mar 28, 2025

Pull request status dashboard

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Mar 28, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 28, 2025

@geflx @arj-odoo cherrypicking of pull request #185911 failed.

stdout:

Auto-merging addons/microsoft_calendar/models/calendar.py
Auto-merging addons/microsoft_calendar/models/res_users.py
CONFLICT (content): Merge conflict in addons/microsoft_calendar/models/res_users.py
Auto-merging addons/microsoft_calendar/tests/common.py
Auto-merging addons/microsoft_calendar/tests/test_create_events.py
CONFLICT (content): Merge conflict in addons/microsoft_calendar/tests/test_create_events.py

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@geflx geflx force-pushed the saas-18.1-16.0-microsoft_calendar-dont_sync_existing_events-gdpf-426178-fw branch from 05be68d to c949738 Compare March 28, 2025 10:21
@C3POdoo C3POdoo requested a review from a team March 28, 2025 10:27
Before this commit, customers were facing problems during the first synchronization
related to the abundance of invitations sent through Outlook from previously created
events in Odoo side. This should not happen because most of the time it is not useful
synchronizing events that were already created before starting the synchronization with
Outlook (from feedbacks discussed with customers and internally).

After this commit, we no longer synchronize events that were created before the first
synchronization of any user in a database if we don't find any token created in it (i.e.
if no user synchronized its Odoo Calendar with Outlook before this improvement). In case
of any user already have synchronized its calendar with Outlook, we won't change the
synchronization behavior (since the biggest part of the invitations were already sent).

task-4294884

X-original-commit: a4d605b
@geflx geflx force-pushed the saas-18.1-16.0-microsoft_calendar-dont_sync_existing_events-gdpf-426178-fw branch from c949738 to 0c041f2 Compare March 28, 2025 10:29
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 28, 2025
@geflx
Copy link
Contributor

geflx commented Mar 28, 2025

@robodoo r+

robodoo pushed a commit that referenced this pull request Mar 28, 2025
Before this commit, customers were facing problems during the first synchronization
related to the abundance of invitations sent through Outlook from previously created
events in Odoo side. This should not happen because most of the time it is not useful
synchronizing events that were already created before starting the synchronization with
Outlook (from feedbacks discussed with customers and internally).

After this commit, we no longer synchronize events that were created before the first
synchronization of any user in a database if we don't find any token created in it (i.e.
if no user synchronized its Odoo Calendar with Outlook before this improvement). In case
of any user already have synchronized its calendar with Outlook, we won't change the
synchronization behavior (since the biggest part of the invitations were already sent).

task-4294884

closes #203896

X-original-commit: a4d605b
Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
Signed-off-by: Gabriel de Paula Felix (gdpf) <gdpf@odoo.com>
@robodoo robodoo closed this Mar 28, 2025
@fw-bot fw-bot deleted the saas-18.1-16.0-microsoft_calendar-dont_sync_existing_events-gdpf-426178-fw branch April 4, 2025 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants