Wednesday, September 19, 2018
2 changes · master
Resolved issues and error corrections
This fix corrects a format conversion issue that prevented users from creating journals. After the change, journals can be created successfully and their related sequence is generated as expected.
Original PR description
Description of the issue/feature this PR addresses: Fix error in bad use of converting formats Current behavior before PR: Cant create a journal Desired behavior after PR is merged: Allow create journal and its sequence is created -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This change stops Odoo from automatically installing a generic accounting setup when a specific country localization module is already queued for installation. It helps new deployments, especially on Odoo.sh, start with the intended local accounting rules instead of requiring manual cleanup or reconfiguration.
Original PR description
# Steps 1. Create an odoo.sh configuration 2. List of modules to install use: l10n_mx  #…
# Steps 1. Create an odoo.sh configuration 2. List of modules to install use: l10n_mx  # Saw generic_coa is installed. # Expected generic_coa not installed since that we want to use l10n_mx and odoo.sh doesn't allow us to choose a `company.country` early. Currently, this behavior is working fine if you have the option to choose the country from the startup wizard -  - In order to auto-install the l10n_COUNTRY: - https://github.com/odoo/odoo/blob/f2b8caff07ba41b851ae9d5015e317753f3eb1a1/addons/account/__init__.py#L16 But odoo.sh settings doesn't have the option to choose a country. Another option is to use a Environment Variable to set early the company country from [company_country](https://github.com/Vauxoo/server-tools/blob/bf96d2cfce812a96da99956ad6a188401c7d98cc/company_country/models/res_config.py#L16) module, but I don't found an option to set environment variables from odoo.sh Another option is to install manually all modules (first changing the company.country) for each dev deploy, but it is consuming time. This PR avoid auto-install l10n_** module if in the list of modules to install there are one or more l10n_* modules. What do you think? @qdp-odoo @antonylesuisse @mart-e @beledouxdenis