-
-
Notifications
You must be signed in to change notification settings - Fork 866
[14.0][MIG] bi_sql_editor #518
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.
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
Conversation
…ield; [ADD] security
…fresh date time on the action name of each sql materialized view
Fix error '... is not a table or foreign table' Fix view (colors + oe_highlight) Fix tests Fix README + manifest Fix back to draft Fix cron call + default values Use Postgres version 9.6 for travis builds
* [IMP] is_materialized field non readonly on sql_valid state ; [FIX] block possibility to set indexes on non materialized view * [FIX] set domain_force, group_ids readonly if state > sql_valid * [IMP] better display of the field group_ids * [IMP] possibility to reorder menu items from sql views * [IMP] Do not warn user when setting sql view to draft if state is sql_valid * [REF] * [FIX] Set Date of the first execution in the action name
[UPD] Update bi_sql_editor.pot [UPD] Update bi_sql_editor.pot [UPD] Update bi_sql_editor.pot
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
bi_sql_editor 12.0.1.1.0 [UPD] README.rst [UPD] README.rst
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: reporting-engine-12.0/reporting-engine-12.0-bi_sql_editor Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-12-0/reporting-engine-12-0-bi_sql_editor/
Currently translated at 29.1% (32 of 110 strings) Translation: reporting-engine-12.0/reporting-engine-12.0-bi_sql_editor Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-12-0/reporting-engine-12-0-bi_sql_editor/de/
richard-willdooit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks complete, to me.
871c1ca to
c807f61
Compare
|
Hi @davejames First thanks a lot for porting this module. could you include some improvement done on V12 and that are not merged, by cherry pick ?
thanks ! |
…zed view to draft ; Set correct default values for cron and enable it
c807f61 to
4648ff1
Compare
[UPD] README.rst
Currently translated at 38.3% (43 of 112 strings) Translation: reporting-engine-13.0/reporting-engine-13.0-bi_sql_editor Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-13-0/reporting-engine-13-0-bi_sql_editor/es/
this commit allows specifying the group operator for each field. This functionality was originally included in a seperate module "bi_sql_editor_aggregate" in version 12.
4648ff1 to
53dc46e
Compare
|
I have cherry-picked 3 out of the 4, however #496 depends on changes in OCA/server-tools#2061 which are not yet merged, let alone ported to 14.0. So I have excluded that one for now. I suggest we open a new PR for it once it is merged and ported to 14.0 |
|
Hi @davejames. Thanks for cherry picking. thanks for porting this module ! |
|
@legalsylvain would it be possible for you to review/merge? |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 93f8832. Thanks a lot for contributing to OCA. ❤️ |
migration to v14
Also includes the functionality to choose a group operator for each field. This functionality was originally written as a seperate module bi_sql_editor_aggregate for version 12 because upgrading existing databases was problematic, however as this is a fresh migration the same problem will not apply so it is included in the base module.