Skip to content

Conversation

@cuongnmtm
Copy link

No description provided.

@mmequignon
Copy link
Member

Hello @cuongnmtm, thanks for this!
Looks like we are missing the git history here.
To fix that, you can either do the migration by hand, following those steps https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0
Or use the oca-port tool that you can find here https://github.com/OCA/oca-port

@cuongnmtm cuongnmtm force-pushed the 14.0-mig-base_export_async branch from 98283d7 to cf7a38e Compare January 11, 2023 04:15
@cuongnmtm cuongnmtm changed the title [MIG] base_export_async: Migration to 14.0 [14.0][MIG] base_export_async: Migration to 14.0 Jan 11, 2023
@cuongnmtm cuongnmtm mentioned this pull request Jan 11, 2023
9 tasks
@cuongnmtm
Copy link
Author

@mmequignon Thanks for your great information. This base_export_async module is copied from 12.0 to 14.0, it is not installable since 13.0. I see the commit history of the module is already there, so I think I don't need to add it to this PR, right?
However, I could apply your suggestion for #493.

@cuongnmtm cuongnmtm force-pushed the 14.0-mig-base_export_async branch 2 times, most recently from c6bb856 to 141988b Compare January 30, 2023 09:58
Copy link

@vincent-cowboy vincent-cowboy left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for the PR

@guewen
Copy link
Member

guewen commented Mar 10, 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 14.0-ocabot-merge-pr-492-by-guewen-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Mar 10, 2023
Signed-off-by guewen
@OCA-git-bot
Copy link
Contributor

@guewen your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-492-by-guewen-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

Guewen Baconnier and others added 10 commits March 16, 2023 18:07
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.
We cannot change the create_date of a record in the test, so it fails.
Change the current time past the TTL instead.
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/
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/
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/
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/
@cuongnmtm cuongnmtm force-pushed the 14.0-mig-base_export_async branch from c6b4499 to 7a817d3 Compare March 16, 2023 11:09
@cuongnmtm cuongnmtm force-pushed the 14.0-mig-base_export_async branch from 7a817d3 to 6deeea9 Compare March 16, 2023 11:19
@cuongnmtm
Copy link
Author

@guewen I have just rebased 14.0 to my branch. Hopefully, it will solve the failed check(s).

@guewen
Copy link
Member

guewen commented Mar 16, 2023

Thanks!

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-492-by-guewen-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5a66c11 into OCA:14.0 Mar 16, 2023
@OCA-git-bot
Copy link
Contributor

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