Skip to content

[18.0][MIG][PATCH] contract_sale #6

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

Open
wants to merge 32 commits into
base: 18.0
Choose a base branch
from
Open

Conversation

CRogos
Copy link

@CRogos CRogos commented May 21, 2025

Waiting for OCA#1138

Luis M. Ontalba and others added 30 commits November 4, 2024 15:12
* README by fragments
* Adjusted dependency
* Changed root menu XML-ID
[REF] Contract Sale: update translations
The lowest model access for contract.contract model is group_account_invoice group. However the addon adds two smart buttons to res.partner view without any security restrictions and without compute_sudo attribute on computed fields.
This causes the view to crash when a user without the proper permissions tries to access the res.partner form view.
The solution adds groups_id to the partner form views in which the buttons are added, so the only loads when the user has proper permissions.
Other way to solve it would be to add compute_sudo attribute to the relevant fields, but this causes an access error when the user clicks on the smart buttons.
Currently translated at 100.0% (1 of 1 strings)

Translation: contract-12.0/contract-12.0-contract_sale
Translate-URL: https://translation.odoo-community.org/projects/contract-12-0/contract-12-0-contract_sale/es_CL/
weblate and others added 2 commits November 4, 2024 15:12
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: contract-17.0/contract-17.0-contract_sale
Translate-URL: https://translation.odoo-community.org/projects/contract-17-0/contract-17-0-contract_sale/
@CRogos CRogos force-pushed the 18.0-mig-contract_sale branch from b922fce to 7184def Compare May 21, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.