Skip to content

[19.0][MIG] sql_request_abstract: Migration to 19.0#1101

Merged
OCA-git-bot merged 104 commits intoOCA:19.0from
adhoc-dev:19.0-mig-sql_request_abstract
Feb 2, 2026
Merged

[19.0][MIG] sql_request_abstract: Migration to 19.0#1101
OCA-git-bot merged 104 commits intoOCA:19.0from
adhoc-dev:19.0-mig-sql_request_abstract

Conversation

@lef-adhoc
Copy link
Contributor

No description provided.

legalsylvain and others added 30 commits November 28, 2025 10:57
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_request_abstract/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_request_abstract/
Currently translated at 100.0% (21 of 21 strings)

Translation: server-tools-12.0/server-tools-12.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_request_abstract/zh_CN/
Currently translated at 100.0% (21 of 21 strings)

Translation: server-tools-12.0/server-tools-12.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_request_abstract/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-12.0/server-tools-12.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-sql_request_abstract/
oca-ci and others added 7 commits November 28, 2025 10:57
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-18.0/reporting-engine-18.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-sql_request_abstract/
Currently translated at 100.0% (47 of 47 strings)

Translation: reporting-engine-18.0/reporting-engine-18.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-sql_request_abstract/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: reporting-engine-18.0/reporting-engine-18.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-sql_request_abstract/
Currently translated at 100.0% (46 of 46 strings)

Translation: reporting-engine-18.0/reporting-engine-18.0-sql_request_abstract
Translate-URL: https://translation.odoo-community.org/projects/reporting-engine-18-0/reporting-engine-18-0-sql_request_abstract/tr/
@adnankkapasi
Copy link

@lef-adhoc Is this PR ready for testing?

<field name="category_id" ref="category_sql_abstract" />
<field name="users" eval="[(4, ref('base.user_admin'))]" />
<field name="privilege_id" ref="privilege_sql_request" />
<field name="user_ids" eval="[(4, ref('base.user_admin'))]" />
Copy link
Contributor

Choose a reason for hiding this comment

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

<field name="user_ids" eval="[Command.link(ref('base.user_admin'))]" />

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know if it's necessary...

Copy link
Contributor

Choose a reason for hiding this comment

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

It's not necessary, but it's a modern Odoo way and actually having a command name aka link is much more clear to understand than old style 4, ID

Copy link

Choose a reason for hiding this comment

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

you can remove this file

Remove references in CREDITS.rst of past financed migrations by other companies.

the request has to be cleaned, checked and tested. All of this
operations can be disabled in the inherited modules.

- This module two new groups:
Copy link

Choose a reason for hiding this comment

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

Suggested change
- This module two new groups:
- This module adds two new groups:

@@ -0,0 +1,20 @@
This module provides an abstract model to manage SQL Select requests on
database. It is not usefull for itself. You can see an exemple of
Copy link

Choose a reason for hiding this comment

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

Suggested change
database. It is not usefull for itself. You can see an exemple of
database. It is not usefull for itself. You can see an example of

Comment on lines +7 to +10
<record model="res.groups.privilege" id="privilege_sql_request">
<field name="name">SQL Request</field>
<field name="category_id" ref="category_sql_abstract" />
</record>
Copy link

Choose a reason for hiding this comment

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

💡 TIL res.groups.privilege

@yankinmax
Copy link
Contributor

Hello @lef-adhoc , can you pls address the suggestions and finish the migration?

@lef-adhoc lef-adhoc force-pushed the 19.0-mig-sql_request_abstract branch from 8d94edc to 082405b Compare February 2, 2026 14:56
@lef-adhoc
Copy link
Contributor Author

@yankinmax Done!

@yankinmax
Copy link
Contributor

yankinmax commented Feb 2, 2026

Hello @HviorForgeFlow @pedrobaeza
Sorry, pinging you directly.
Can we probably merge this migration PR?

@pedrobaeza
Copy link
Member

/ocabot migration sql_request_abstract

I don't know about this module, sorry.

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Feb 2, 2026
Copy link
Member

@HviorForgeFlow HviorForgeFlow left a comment

Choose a reason for hiding this comment

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

Code review 👍

@HviorForgeFlow
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 626770a into OCA:19.0 Feb 2, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@lef-adhoc lef-adhoc deleted the 19.0-mig-sql_request_abstract branch February 2, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.