Skip to content

[16.0][IMP] mail_tracking: display internal notifications status#1331

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
xcgd:16.0-mail_tracking-imp-display-inbox-mail-notification-status
Nov 25, 2024
Merged

[16.0][IMP] mail_tracking: display internal notifications status#1331
OCA-git-bot merged 1 commit intoOCA:16.0from
xcgd:16.0-mail_tracking-imp-display-inbox-mail-notification-status

Conversation

@vincent-hatakeyama
Copy link
Copy Markdown

Use internal notifications status when mails are not generated.

hbrunn
hbrunn previously approved these changes Mar 19, 2024
Copy link
Copy Markdown
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please rebase

Comment thread mail_tracking/models/mail_message.py
Comment thread mail_tracking/models/mail_message.py Outdated
@hbrunn hbrunn self-requested a review March 19, 2024 22:28
@vincent-hatakeyama vincent-hatakeyama force-pushed the 16.0-mail_tracking-imp-display-inbox-mail-notification-status branch from 2ad02d1 to 3673de3 Compare March 20, 2024 14:18
@vincent-hatakeyama vincent-hatakeyama force-pushed the 16.0-mail_tracking-imp-display-inbox-mail-notification-status branch from 3673de3 to 894d213 Compare March 25, 2024 11:08
Use internal notifications status when mails are not generated
@vincent-hatakeyama vincent-hatakeyama force-pushed the 16.0-mail_tracking-imp-display-inbox-mail-notification-status branch from 894d213 to 907f4e9 Compare July 22, 2024 09:09
@github-actions
Copy link
Copy Markdown

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 Nov 24, 2024
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge patch

Please fw-port it to upper versions.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1331-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

@pedrobaeza The merge process could not be finalized, because command git push origin 16.0-ocabot-merge-pr-1331-by-pedrobaeza-bump-patch:16.0 failed with output:

To https://github.com/OCA/social
 ! [rejected]          16.0-ocabot-merge-pr-1331-by-pedrobaeza-bump-patch -> 16.0 (fetch first)
error: failed to push some refs to 'https://github.com/OCA/social'
hint: Updates were rejected because the remote contains work that you do not
hint: have locally. This is usually caused by another repository pushing to
hint: the same ref. If you want to integrate the remote changes, use
hint: 'git pull' before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@OCA-git-bot OCA-git-bot merged commit 7c892b1 into OCA:16.0 Nov 25, 2024
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at fb34dbb. Thanks a lot for contributing to OCA. ❤️

@vincent-hatakeyama vincent-hatakeyama deleted the 16.0-mail_tracking-imp-display-inbox-mail-notification-status branch July 1, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved merged 🎉 stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants