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

[18.0][MIG] sql_export #967

Merged
merged 91 commits into from
Jan 22, 2025
Merged

[18.0][MIG] sql_export #967

merged 91 commits into from
Jan 22, 2025

Conversation

bealdav
Copy link
Member

@bealdav bealdav commented Jan 17, 2025

No description provided.

florian-dacosta and others added 30 commits January 17, 2025 14:22
@florian-dacosta @bealdav
Add sql export module (migration from v7)
da33469 
Add rollback after executing query as a double security with blacklist terms

add known issue in readme
@bguillot @bealdav
[IMP] add support of variables in queries
3a38ea5 
[FIX] encoding with mogrify

[FIX] hide placeholder
@mourad-ehm @bealdav
[MIG] Migrate module from 8.0 to 9.0
699c237
@legalsylvain @bealdav
[REF] create a new module sql_request_abstract
8c5e66a
@florian-dacosta @bealdav
Fix version
063b9df
@AaronHForgeFlow @bealdav
[IMP]encode output file
1883be6
@legalsylvain @bealdav
[FIX] various improvement. Courtesy @dreispt review
971f249
@oca-transbot @bealdav
OCA Transbot updated translations from Transifex
e1309c0
@AaronHForgeFlow @bealdav
[MIG][10.0]sql_export v10
bf6bc65
@MiquelRForgeFlow @bealdav
[FIX] mogrify doesn't allow dicts
2a64ef6
@oca-travis @bealdav
[UPD] Update sql_export.pot
d9c6365
@rschnapka @bealdav
Translated using Weblate (German)
d7dec55 
Currently translated at 100,0% (50 of 50 strings)

Translation: server-tools-10.0/server-tools-10.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-10-0/server-tools-10-0-sql_export/de/
@bealdav
IMP Readme for Sql Export Module
f8b1b7c
@oca-transbot @bealdav
Update translation files
211b3d1 
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@bealdav
PORT v12 sql_export
4391002
@bealdav
FIX sql_export: test and misc
a0201e1
@oca-travis @bealdav
[UPD] Update sql_export.pot
7c132ef
@sbidoul @bealdav
[FIX] title level in readme fragments
9ad839d
@OCA-git-bot @bealdav
[UPD] README.rst
82eb7c5
@oca-transbot @bealdav
Update translation files
226c4e6 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_export/
@bealdav
IMP sql_export: render field as require when matching field def
1903d49
@florian-dacosta @bealdav
Fix uses of params in sql query
6e8695d
@OCA-git-bot @bealdav
[UPD] README.rst
4039165
@florian-dacosta @bealdav
Adapt module sql_export_mail to be compatible with multiple file format
874aca5
@liweijie0812 @bealdav
Translated using Weblate (Chinese (Simplified))
1153129 
Currently translated at 100.0% (49 of 49 strings)

Translation: server-tools-12.0/server-tools-12.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_export/zh_CN/
@florian-dacosta @bealdav
Migrate sql_export_excel to v12
403d9e6
@bealdav
Translated using Weblate (Portuguese (Brazil))
576be8e 
Currently translated at 100.0% (49 of 49 strings)

Translation: server-tools-12.0/server-tools-12.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_export/pt_BR/
@florian-dacosta @bealdav
Small fixes for v12/python3 standard
f3df290 
Co-Authored-By: David Beal <david.beal@akretion.com>
@oca-travis @bealdav
[UPD] Update sql_export.pot
111faf4
@oca-transbot @bealdav
Update translation files
97ccfa4 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_export/
ibuioli and others added 10 commits January 17, 2025 14:22
@ibuioli @bealdav
Translated using Weblate (Spanish (Argentina))
2cef1be 
Currently translated at 100.0% (64 of 64 strings)

Translation: reporting-engine-16.0/reporting-engine-16.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-sql_export/es_AR/
@florian-dacosta @bealdav
[FIX] sql_export : remove unused action parameter
903cd3b
@OCA-git-bot @bealdav
[BOT] post-merge updates
f48b4c6
@mymage @bealdav
Translated using Weblate (Italian)
53ccda0 
Currently translated at 23.4% (15 of 64 strings)

Translation: reporting-engine-16.0/reporting-engine-16.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-16-0/reporting-engine-16-0-sql_export/it/
@sanderlienaerts @bealdav
[IMP] sql_export: black, isort, prettier
cade6cd
@sanderlienaerts @bealdav
[MIG] sql_export: Migration to 17.0
f45fa52
@bealdav
[UPD] Update sql_export.pot
8842cf8
@mymage @bealdav
Translated using Weblate (Italian)
338ba0d 
Currently translated at 100.0% (64 of 64 strings)

Translation: reporting-engine-17.0/reporting-engine-17.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-sql_export/it/
@ibuioli @bealdav
Translated using Weblate (Spanish (Argentina))
fc902c8 
Currently translated at 100.0% (64 of 64 strings)

Translation: reporting-engine-17.0/reporting-engine-17.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-17-0/reporting-engine-17-0-sql_export/es_AR/
@bealdav
[IMP] sql_export: pre-commit execution
96439ef
@bealdav bealdav force-pushed the 18.0-mig-sql_export branch from cb7134e to ca23e80 Compare January 17, 2025 14:14
@bealdav bealdav marked this pull request as ready for review January 17, 2025 14:18
florian-dacosta
florian-dacosta approved these changes Jan 17, 2025
View reviewed changes
Copy link
Contributor

@florian-dacosta florian-dacosta left a comment

Choose a reason for hiding this comment

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

LGTM code review

marcelsavegnago
marcelsavegnago approved these changes Jan 17, 2025
View reviewed changes
Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

@bealdav bealdav force-pushed the 18.0-mig-sql_export branch from ca23e80 to 8b4eb72 Compare January 20, 2025 16:40
@bealdav bealdav force-pushed the 18.0-mig-sql_export branch from 8b4eb72 to e387ac8 Compare January 20, 2025 16:46
@bealdav
Copy link
Member Author

bealdav commented Jan 22, 2025

Hi @lmignon @HviorForgeFlow @etobella please could you merge ?

Thanks a lot

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

@rvalyi
Copy link
Member

rvalyi commented Jan 22, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 8576271 into OCA:18.0 Jan 22, 2025
5 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 019b15a. Thanks a lot for contributing to OCA. ❤️

@bealdav bealdav deleted the 18.0-mig-sql_export branch January 22, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rvalyi rvalyi rvalyi approved these changes

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@florian-dacosta florian-dacosta florian-dacosta 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.