Skip to content

[16.0][MIG] sql_export_mail (from OCA/server-tools 14.0) #672

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
Merged
OCA-git-bot merged 22 commits into from
Jan 9, 2023
Merged

[16.0][MIG] sql_export_mail (from OCA/server-tools 14.0) #672

OCA-git-bot merged 22 commits into from
Jan 9, 2023

Conversation

@legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Oct 26, 2022 •
edited
Loading

simple PR

  • Update views, regarding changes in sql_export module
  • Add prepare function for cron vals

depends on #671, #669

@legalsylvain legalsylvain changed the title [16.0][MIG] sql_export_mail [16.0][MIG] sql_export_mail (from OCA/server-tools 14.0) Oct 26, 2022
@legalsylvain
Copy link
Contributor Author

legalsylvain commented Oct 26, 2022

/ocabot migration sql_export_mail

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Oct 26, 2022

Sorry @legalsylvain you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@HviorForgeFlow
Copy link
Member

HviorForgeFlow commented Oct 26, 2022

/ocabot migration sql_export_mail

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Oct 26, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 26, 2022
Open
15 tasks
@florian-dacosta
Copy link
Contributor

florian-dacosta commented Nov 28, 2022

@legalsylvain can you rebase please?

@legalsylvain
Copy link
Contributor Author

legalsylvain commented Nov 28, 2022

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Nov 28, 2022

Sorry @legalsylvain you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@florian-dacosta
Copy link
Contributor

florian-dacosta commented Dec 5, 2022

@legalsylvain Rebase did not work, could you do it without oca bot ?
Thanks !

florian-dacosta and others added 20 commits December 5, 2022 15:34
@florian-dacosta @legalsylvain
Add module sql_export_mail
92f4ea0
@florian-dacosta @legalsylvain
Remove useless files
6dd45ce
@florian-dacosta @legalsylvain
styles and decorator fixes
d3c422a
@florian-dacosta @legalsylvain
Migrate sql_export_mail to v9
93fdd36
@florian-dacosta @legalsylvain
Migrate to v12
94f864c
@florian-dacosta @legalsylvain
Add group on cron field to avoid access error when user does not have…
9c88957 
… the right to read crons.
@florian-dacosta @legalsylvain
Adapt module sql_export_mail to be compatible with multiple file format
65b9099
@oca-travis @legalsylvain
[UPD] Update sql_export_mail.pot
9caed14
@OCA-git-bot @legalsylvain
[UPD] README.rst
99cfe0f
@OCA-git-bot @legalsylvain
[ADD] icon.png
991be25
@oca-transbot @legalsylvain
Update translation files
1dedee2 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_export_mail
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_export_mail/
@hkapatel-initos @legalsylvain
[IMP] sql_export_mail: black, isort, prettier
f7585a5
@hkapatel-initos @legalsylvain
[MIG] sql_export_mail: Migration to 14.0
647d954
@oca-travis @legalsylvain
[UPD] Update sql_export_mail.pot
584fd6b
@OCA-git-bot @legalsylvain
[UPD] README.rst
e54dd17
@oca-transbot @legalsylvain
Update translation files
7ac221c 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-14.0/server-tools-14.0-sql_export_mail
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-sql_export_mail/
@oca-travis @legalsylvain
[UPD] Update sql_export_mail.pot
e68cf03
@oca-transbot @legalsylvain
Update translation files
d9adef3 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-14.0/server-tools-14.0-sql_export_mail
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-sql_export_mail/
@oca-travis @legalsylvain
[UPD] Update sql_export_mail.pot
a1b16c3
@oca-transbot @legalsylvain
Update translation files
668ea5c 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-14.0/server-tools-14.0-sql_export_mail
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-sql_export_mail/
ofonseca-pyming and others added 2 commits December 5, 2022 15:34
@ofonseca-pyming @legalsylvain
Added translation using Weblate (Catalan)
12d6704
@legalsylvain
[MIG] sql_export_mail from OCA/server-tools 14.0 to OCA/reporting-eng…
c62e8f2 
…ine 16.0 branch

- Update views, regarding changes in sql_export module
- Add prepare function for cron vals
@legalsylvain legalsylvain force-pushed the 16.0-mig-sql_export_mail branch from 1101fcc to c62e8f2 Compare December 5, 2022 14:35
@legalsylvain
Copy link
Contributor Author

legalsylvain commented Dec 5, 2022

Done.
CI looks green.

bealdav
bealdav approved these changes Jan 4, 2023
View reviewed changes
Copy link
Member

@bealdav bealdav left a comment

Choose a reason for hiding this comment

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

LGTM

@HviorForgeFlow
Copy link
Member

HviorForgeFlow commented Jan 9, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 9, 2023

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-672-by-HviorForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5a1668d into OCA:16.0 Jan 9, 2023
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 9, 2023

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

2 more reviewers

@bealdav bealdav bealdav approved these changes

@florian-dacosta florian-dacosta florian-dacosta approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉

Projects

None yet

Milestone

16.0

Development

Successfully merging this pull request may close these issues.

9 participants

@legalsylvain @OCA-git-bot @HviorForgeFlow @florian-dacosta @bealdav @oca-travis @oca-transbot @hkapatel-initos @ofonseca-pyming