Skip to content

[18.0][MIG] web_notify_channel_message - #3026

Merged
OCA-git-bot merged 21 commits into
OCA:18.0from
ForgeFlow:18.0-mig-web_notify_channel_message
Mar 21, 2025
Merged

OCA-git-bot merged 21 commits into
OCA:18.0from
ForgeFlow:18.0-mig-web_notify_channel_message

Conversation

@JasminSForgeFlow

Copy link
Copy Markdown
Contributor

Standard Migration

@ForgeFlow

JordiBForgeFlow and others added 16 commits December 19, 2024 10:30
Currently translated at 100.0% (5 of 5 strings)

Translation: web-14.0/web-14.0-web_notify_channel_message
Translate-URL: https://translation.odoo-community.org/projects/web-14-0/web-14-0-web_notify_channel_message/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-16.0/web-16.0-web_notify_channel_message
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_notify_channel_message/
Currently translated at 100.0% (2 of 2 strings)

Translation: web-16.0/web-16.0-web_notify_channel_message
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_notify_channel_message/es/
Currently translated at 100.0% (2 of 2 strings)

Translation: web-16.0/web-16.0-web_notify_channel_message
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_notify_channel_message/it/
If other user than the admin is trying to send a message to a channel it will show an error message. Sending it as sudo solves the problem and makes sense that in this case you will always want to send the notification.
@ThiagoMForgeFlow

Copy link
Copy Markdown
Contributor

Add recent change to history #2982

@JasminSForgeFlow
JasminSForgeFlow force-pushed the 18.0-mig-web_notify_channel_message branch from 4db38f4 to 12f9baf Compare February 3, 2025 05:40

@ThiagoMForgeFlow ThiagoMForgeFlow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@DavidJForgeFlow

Copy link
Copy Markdown
Contributor

@pedrobaeza can you take a look on this? Thanks!

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration web_notify_channel_message

Merging due to the existing reviews:

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 21, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-3026-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Mar 21, 2025
49 tasks
@OCA-git-bot
OCA-git-bot merged commit caa4d87 into OCA:18.0 Mar 21, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants