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

[17.0][MIG] mail_optional_follower_notification: Migration to 17.0 #1303

Conversation

beagle-cloud
Copy link

No description provided.

SodexisTeam and others added 30 commits November 21, 2023 18:35
Signed-off-by dreispt
Introduce search of html fields
Create indexes
Update Readme
On views with a searchpanel, xpath `//field[last()]` may select an element inside
the search panel, provoking the error "Only types ['many2one'] are supported for
category (found type text)".

This fix uses an stricter xpath for avoiding this situation.

TT25796
Avoid CacheMiss errors setting a value on message_content field.
OCA-git-bot and others added 19 commits February 8, 2024 15:00
Currently translated at 50.0% (2 of 4 strings)

Translation: social-14.0/social-14.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-14-0/social-14-0-mail_optional_follower_notification/es/
Currently translated at 25.0% (1 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/it/
Currently translated at 100.0% (4 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/pt_BR/
@beagle-cloud beagle-cloud changed the title [17.0] [MIG] mail_optional_follower_notification: Migration to 17.0 [17.0][MIG] mail_optional_follower_notification: Migration to 17.0 Feb 9, 2024
@beagle-cloud beagle-cloud mentioned this pull request Feb 9, 2024
30 tasks
@etobella
Copy link
Member

Can you rebase? we needed to apply a change on main branch

@hbrunn
Copy link
Member

hbrunn commented Feb 19, 2024

/ocabot rebase
/ocabot migration mail_optional_follower_notification

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Feb 19, 2024
@OCA-git-bot
Copy link
Contributor

@hbrunn The rebase process failed, because command git rebase origin/17.0 failed with output:

First, rewinding head to replay your work on top of it...
Applying: [IMP] mail_outbound_static: pre-commit auto fixes
.git/rebase-apply/patch:284: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
M	mail_outbound_static/README.rst
M	mail_outbound_static/models/ir_mail_server.py
A	mail_outbound_static/readme/CONTRIBUTORS.rst
A	mail_outbound_static/readme/DESCRIPTION.rst
A	mail_outbound_static/readme/ROADMAP.rst
A	mail_outbound_static/readme/USAGE.rst
M	mail_outbound_static/static/description/index.html
M	mail_outbound_static/tests/test_ir_mail_server.py
Falling back to patching base and 3-way merge...
Auto-merging mail_outbound_static/static/description/index.html
Auto-merging mail_outbound_static/models/ir_mail_server.py
Auto-merging mail_outbound_static/README.rst
CONFLICT (content): Merge conflict in mail_outbound_static/README.rst
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch' to see the failed patch
Patch failed at 0001 [IMP] mail_outbound_static: pre-commit auto fixes
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".

@llabusch93
Copy link
Contributor

Hey,

I tested it and it does not work. All followers get notified, no matter what.

@pedrobaeza
Copy link
Member

Superseded by #1381

@pedrobaeza pedrobaeza closed this Jun 4, 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.

None yet