Skip to content

[15.0][MIG] sql_export #553

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 50 commits into from
Feb 1, 2022
Merged

[15.0][MIG] sql_export #553

OCA-git-bot merged 50 commits into from
Feb 1, 2022

Conversation

@mariadforgeflow
Copy link
Contributor

@mariadforgeflow mariadforgeflow commented Dec 2, 2021 •
edited
Loading

Standard Migration to v15.
Moved from server-tools.

@JoanMForgeFlow

legalsylvain
legalsylvain approved these changes Dec 2, 2021
View reviewed changes
JoanMForgeFlow
JoanMForgeFlow approved these changes Dec 2, 2021
View reviewed changes
Copy link
Contributor

@JoanMForgeFlow JoanMForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM, Lets get this merged!

@mariadforgeflow mariadforgeflow mentioned this pull request Jan 12, 2022
Closed
20 tasks
This was referenced Jan 31, 2022
Closed
Closed
@sebalix sebalix added this to the 15.0 milestone Jan 31, 2022
@sebalix
Copy link
Contributor

sebalix commented Jan 31, 2022

Build is failing, can you check?

@yankinmax
Copy link
Contributor

yankinmax commented Feb 1, 2022 •
edited
Loading

@mariadforgeflow can you pls check and include those changes from my PR?
OCA/server-tools@2ce3594
OCA/server-tools@0198d0c
OCA/server-tools@327697f
UPDATED
or you can merge this PR and include changes to fix the build
ForgeFlow#3

@mariadforgeflow
Copy link
Contributor Author

mariadforgeflow commented Feb 1, 2022

@yankinmax I think that the best way to proceed is to first merge the migration PR and then open a new one with improvements. I will now include the changes to fix the build:)

@yankinmax
Copy link
Contributor

yankinmax commented Feb 1, 2022

@mariadforgeflow as you wish, I'm here only to help to get it merged ASAP.
But, as you can see, you can cherry-pick my migration commit and it will be faster.
Thanks for quick response.

florian-dacosta and others added 17 commits February 1, 2022 09:50
@florian-dacosta @mariadforgeflow
Add sql export module (migration from v7)
b6c46ee 
Add rollback after executing query as a double security with blacklist terms

add known issue in readme
@bguillot @mariadforgeflow
[IMP] add support of variables in queries
cd8db95 
[FIX] encoding with mogrify

[FIX] hide placeholder
@mourad-ehm @mariadforgeflow
[MIG] Migrate module from 8.0 to 9.0
97a19be
@legalsylvain @mariadforgeflow
[REF] create a new module sql_request_abstract
c8743f4
@florian-dacosta @mariadforgeflow
Fix version
1ae2161
@AaronHForgeFlow @mariadforgeflow
[IMP]encode output file
1745245
@legalsylvain @mariadforgeflow
[FIX] various improvement. Courtesy @dreispt review
b4c6f3f
@oca-transbot @mariadforgeflow
OCA Transbot updated translations from Transifex
a18b981
@AaronHForgeFlow @mariadforgeflow
[MIG][10.0]sql_export v10
2f8ae9a
@MiquelRForgeFlow @mariadforgeflow
[FIX] mogrify doesn't allow dicts
0e8c991
@oca-travis @mariadforgeflow
[UPD] Update sql_export.pot
2f43eef
@rschnapka @mariadforgeflow
Translated using Weblate (German)
9309e9d 
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 @mariadforgeflow
IMP Readme for Sql Export Module
7afe805
@oca-transbot @mariadforgeflow
Update translation files
6b7504d 
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@bealdav @mariadforgeflow
PORT v12 sql_export
a699d93
@bealdav @mariadforgeflow
FIX sql_export: test and misc
f97729a
@oca-travis @mariadforgeflow
[UPD] Update sql_export.pot
cdb8454
oca-transbot and others added 18 commits February 1, 2022 09:50
@oca-transbot @mariadforgeflow
Update translation files
6f57108 
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/
@fernandahf @mariadforgeflow
[FIX] sql_export: fix warning two fields have same label
50e12f1
@oca-travis @mariadforgeflow
[UPD] Update sql_export.pot
caacf39
@OCA-git-bot @mariadforgeflow
sql_export 12.0.1.1.1
2c3b17d
@oca-transbot @mariadforgeflow
Update translation files
97915ef 
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/
@florian-dacosta @mariadforgeflow
[IMP] sql_export: black, isort, prettier
8f04c73
@florian-dacosta @mariadforgeflow
Migrate to 14.0
8b863e6
@florian-dacosta @mariadforgeflow
Update roadmap for v15 and make some usability improvements
db29738 
Improve usability about variable creation and choice by hiding base wizard base fields and display simplified view
Add a description on paramter fields to help the user to use parameters
If not parameter are used for the query, display the wizard with the generated file directly
@oca-travis @mariadforgeflow
[UPD] Update sql_export.pot
3d8e41d
@OCA-git-bot @mariadforgeflow
[UPD] README.rst
40740aa
@legalsylvain @mariadforgeflow
[PORT] 14.0 port #2061 : sql_export : add preview button
0c7daf1
@oca-travis @mariadforgeflow
[UPD] Update sql_export.pot
d09106a
@OCA-git-bot @mariadforgeflow
sql_export 14.0.1.1.0
3b7f53e
@oca-transbot @mariadforgeflow
Update translation files
71496ce 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

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

Translation: server-tools-14.0/server-tools-14.0-sql_export
Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-sql_export/
@mariadforgeflow
[IMP] sql_export: black, isort, prettier
bc9519c
@mariadforgeflow
[MIG] sql_export: Migration to 15.0
78fa4a4
@mariadforgeflow
Copy link
Contributor Author

mariadforgeflow commented Feb 1, 2022

@yankinmax done:)

JoanMForgeFlow
JoanMForgeFlow approved these changes Feb 1, 2022
View reviewed changes
Copy link
Contributor

@JoanMForgeFlow JoanMForgeFlow left a comment

Choose a reason for hiding this comment

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

green! LG

@simahawk
Copy link
Contributor

simahawk commented Feb 1, 2022

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Feb 1, 2022

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-553-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6b0b63e into OCA:15.0 Feb 1, 2022
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Feb 1, 2022

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

@legalsylvain legalsylvain legalsylvain approved these changes

+3 more reviewers

@leemannd leemannd leemannd approved these changes

@yankinmax yankinmax yankinmax approved these changes

@JoanMForgeFlow JoanMForgeFlow JoanMForgeFlow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 migration

Projects

None yet

Milestone

15.0

Development

Successfully merging this pull request may close these issues.