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

[14.0][FIX] mail_activity_team: Manually keep user_id in sync with team_user_id #1353

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

ypapouin
Copy link
Contributor

Ensure that the built-in user_id field value is in sync with our field team_user_id before triggering create, otherwise the framework will try to resolve the inverse value by itself and will call write before create has ended. That leads to multiple side effects like sending two assignment notifications.

mail_activity_team/models/mail_activity.py Show resolved Hide resolved
mail_activity_team/models/mail_activity.py Outdated Show resolved Hide resolved
Copy link

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.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 29, 2024
…user_id`

Ensure that the built-in `user_id` field value is in sync with our field `team_user_id` before triggering `create`, otherwise the framework will try to resolve the inverse value by itself and will call `write` before create has ended.
That leads to multiple side effects like sending two assignment notifications.
@ypapouin ypapouin force-pushed the 14.0-fix-mail_activity_team_user-sync branch from ef2c1f2 to 6a09208 Compare October 7, 2024 13:40
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants