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.

Sign up for GitHub

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

Jump to bottom

[16.0][MIG] base_export_async: Migration to 16.0 #553

Merged
merged 21 commits into from
Jul 24, 2023
Merged

[16.0][MIG] base_export_async: Migration to 16.0 #553

merged 21 commits into from
Jul 24, 2023

Conversation

bencoronel
Copy link
Contributor

No description provided.

guewen and others added 18 commits July 10, 2023 17:56
@guewen
Support several recipient users in base_export_async
af4c4d7 
The UI still send it to the current user, but it will allow
reusing the module for other purposes: plan a regular export
to users.

Setting the recipient_ids instead of the email_to in mail.mail means
that we can still send the email, it will stay in the failed emails,
allowing to correct the user.
@oca-travis
[UPD] Update base_export_async.pot
082f6e7
@OCA-git-bot
[UPD] README.rst
3a6c256
@oca-transbot
Update translation files
2f8ecea 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-12.0/queue-12.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-12-0/queue-12-0-base_export_async/
@ypapouin
Added translation using Weblate (French)
0f07de7
@ypapouin
Translated using Weblate (French)
89686c8 
Currently translated at 100.0% (18 of 18 strings)

Translation: queue-12.0/queue-12.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-12-0/queue-12-0-base_export_async/fr/
@oca-transbot
Update translation files
8bf920a 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-12.0/queue-12.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-12-0/queue-12-0-base_export_async/
@hhgabelgaard
Added translation using Weblate (Danish)
f397c78
@hhgabelgaard
Translated using Weblate (Danish)
378d0e9 
Currently translated at 100.0% (18 of 18 strings)

Translation: queue-12.0/queue-12.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-12-0/queue-12-0-base_export_async/da/
@pedrocs-exo
Added translation using Weblate (Portuguese)
dc54e92
@pedrocs-exo
Translated using Weblate (Portuguese)
b8d1e0f 
Currently translated at 94.4% (17 of 18 strings)

Translation: queue-12.0/queue-12.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-12-0/queue-12-0-base_export_async/pt/
@primes2h
Added translation using Weblate (Italian)
5631b76
@vincent-cowboy
[IMP] base_export_async: black, isort, prettier
5526735
@vincent-cowboy
[MIG] base_export_async: Migration to 15.0
24ef9c3
@vincent-cowboy
[REF] base_export_async: send mail via delay_export_mail_template
52e00dc
[UPD] Update base_export_async.pot
e9b2e9e
@OCA-git-bot
[UPD] README.rst
e33a43f
@weblate
Update translation files
6e4a890 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-15.0/queue-15.0-base_export_async
Translate-URL: https://translation.odoo-community.org/projects/queue-15-0/queue-15-0-base_export_async/
@bencoronel bencoronel changed the title [MIG] base_export_migration: Migration to 16.0 [MIG] base_export_asyncMigration to 16.0 Jul 11, 2023
@bencoronel bencoronel changed the title [MIG] base_export_asyncMigration to 16.0 [MIG] base_export_async: Migration to 16.0 Jul 11, 2023
@kikopeiro kikopeiro mentioned this pull request Jul 11, 2023
Open
8 tasks
@bencoronel bencoronel force-pushed the 16.0-mig-base_export_async branch from a2a2dc1 to 1b1f286 Compare July 11, 2023 10:53
@bencoronel bencoronel changed the title [MIG] base_export_async: Migration to 16.0 [16.0][MIG] base_export_async: Migration to 16.0 Jul 13, 2023
@asierneiradev asierneiradev force-pushed the 16.0-mig-base_export_async branch from e79cce5 to 9f07392 Compare July 17, 2023 07:25
@asierneiradev asierneiradev force-pushed the 16.0-mig-base_export_async branch from 9f07392 to 7368fa8 Compare July 17, 2023 07:26
@asierneiradev
Copy link
Contributor

Hi @guewen @sbidoul @ivantodorovich ,

Someone can take a look at the proposed PR.

Thank you very much,
best regards

@bencoronel bencoronel force-pushed the 16.0-mig-base_export_async branch from a0fb5a9 to 547d84f Compare July 17, 2023 11:39
@bencoronel bencoronel force-pushed the 16.0-mig-base_export_async branch from 547d84f to 6c79381 Compare July 17, 2023 11:55
@kikopeiro
Copy link
Contributor

Hi @vincent-cowboy @amh-mw , could you take a look to this PR?

Thanks!

kikopeiro
kikopeiro approved these changes Jul 18, 2023
View reviewed changes
Copy link
Contributor

@kikopeiro kikopeiro left a comment

Choose a reason for hiding this comment

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

LGTM (without code review)
👍

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@julenfl
Copy link

julenfl commented Jul 24, 2023

Hi @guewen @sbidoul @ivantodorovich ,

Could someone please merge this PR, it seems correct

Thank you

@guewen
Copy link
Member

guewen commented Jul 24, 2023

Thanks for the work!

@guewen
Copy link
Member

guewen commented Jul 24, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-553-by-guewen-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

To https://github.com/OCA/queue
 ! [rejected]        16.0-ocabot-merge-pr-553-by-guewen-bump-nobump -> 16.0 (fetch first)
error: failed to push some refs to 'https://***@github.com/OCA/queue'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., '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 57ac6dd into OCA:16.0 Jul 24, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 111ccd2. 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
Reviewers

@kikopeiro kikopeiro kikopeiro approved these changes

@amh-mw amh-mw amh-mw approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.