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

[MIG] project_task_send_by_mail: Migration to 13.0 #673

Merged
merged 8 commits into from Jun 17, 2020
Merged

[MIG] project_task_send_by_mail: Migration to 13.0 #673

merged 8 commits into from Jun 17, 2020

Conversation

dalonsod
Copy link
Contributor

Standard migration

cubells and others added 8 commits May 18, 2020 12:40
@cubells @dalonsod
project_task_send_by_mail module (OCA#303)
16667fd 
* [ADD][9.0] project_task_send_by_mail module

* Add project_task_report dependency

* Update README instructions and remove useless key

* Update task report reference

* Update travis.yml to use wkhtmltopdf

* Remove wkhtmltopdf references in .travis.yml

* Update .travis.yml
@oca-transbot @dalonsod
OCA Transbot updated translations from Transifex
b1b64db
@cubells @dalonsod
project_task_send_by_mail: Migration to 11.0
7c48cbc
@oca-travis @dalonsod
Update project_task_send_by_mail.pot
f806d3a
@oca-transbot @dalonsod
Update translation files
05b0f3d 
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@Nikul-Chaudhary @dalonsod
[MIG] project_task_send_by_mail v11 to v12
9dae045
@dalonsod
[IMP] project_task_send_by_mail: black, isort, prettier
69b5dd1
@dalonsod
[MIG] project_task_send_by_mail: Migration to 13.0
7a4b50a
@dalonsod dalonsod mentioned this pull request May 18, 2020
Closed
35 tasks
@dalonsod
Copy link
Contributor Author

@Nikul-Chaudhary @cubells could you review? Thanks!!

HaraldPanten
HaraldPanten approved these changes May 18, 2020
View reviewed changes
Copy link

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

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

Functional review 👍

emagdalenaC2i
emagdalenaC2i approved these changes May 18, 2020
View reviewed changes
Copy link
Contributor

@emagdalenaC2i emagdalenaC2i left a comment

Choose a reason for hiding this comment

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

LGTM

@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). 🤖

@dalonsod
Copy link
Contributor Author

@OCA/project-service-maintainers is this PR ready to merge? Thanks!!!!

@dreispt
Copy link
Sponsor Member

dreispt commented Jun 17, 2020

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-673-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 16f027d into OCA:13.0 Jun 17, 2020
@OCA-git-bot
Copy link
Contributor

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

@emagdalenaC2i emagdalenaC2i emagdalenaC2i approved these changes

@HaraldPanten HaraldPanten HaraldPanten 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.

None yet
9 participants
@dalonsod @OCA-git-bot @dreispt @emagdalenaC2i @HaraldPanten @cubells @oca-transbot @oca-travis @Nikul-Chaudhary