-
-
Notifications
You must be signed in to change notification settings - Fork 77
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
[15.0][MIG] account_consolidation_oca #43
Draft
Saran440
wants to merge
48
commits into
OCA:15.0
Choose a base branch
from
ecosoft-odoo:15.0-mig-account_consolidation_oca
base: 15.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
[15.0][MIG] account_consolidation_oca #43
Saran440
wants to merge
48
commits into
OCA:15.0
from
ecosoft-odoo:15.0-mig-account_consolidation_oca
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(lp:c2c-addons/6.1 rev 3) [IMP] account_consolidation: progress in the coding, move and move lines are created at holding level (lp:c2c-addons/6.1 rev 4) [IMP] account_consolidation: improvement of consolidate and demo datas (lp:c2c-addons/6.1 rev 5) [IMP] account_consolidation: progress (lp:c2c-addons/6.1 rev 6) [IMP] account_consolidation: better display of errors in checks, fiscal year field on check instead of periods, tests (lp:c2c-addons/6.1 rev 7) [IMP] commented out gain and loss accounts (lp:c2c-addons/6.1 rev 8) [IMP] account_consolidation: integration with account reversal (lp:c2c-addons/6.1 rev 9) [IMP] account_consolidation : uses the new currency rate type (merge on trunk server is pending on lp:~openerp-dev/openobject-server/trunk-currency_rate_type-mra) to compute average consolidation (lp:c2c-addons/6.1 rev 10) [FIX] account_consolidation: currency conversion from spot (lp:c2c-addons/6.1 rev 11) [MRG] from upstream (lp:c2c-addons/6.1 rev 24.1.2) [FIX] account_consolidation: adaptations to let the wizards run on 6.1 version (lp:c2c-addons/6.1 rev 43.1.1) [FIX] account_consolidation: missing account_reversal dependency (lp:c2c-addons/6.1 rev 43.1.2) [FIX] account_consolidation: signature of create_reversals has been modified in account_reversal (lp:c2c-addons/6.1 rev 43.1.3) [FIX] Remove file that failed to be removed by the extraction script from c2c-financial-addons
[MIGR] set modules not installable (need migration) [IMP] manifest [FIX] xml indentation [FIX] no longer view_id in account.journal [IMP] python formatting and openerp standards [MIGR] migration of views [FIX] years were hardcoded in tests [FIX] february last day may change [FIX] test fiscal years conflicts with demo datas, the misconfigured fy have to be the next year [FIX] xmlid has changed [FIX] unclear docstring [FIX] currency rate type should return an id instead of a browse record, fix the docstring as well
…lues in debit and credit. Somes without any debit and any credit [TYPO] in variable name [ADD] implementation of consolidation difference management [FIX] indentation error [FIX] empty moves are generated during consolidation process [ADD] difference line label [ADD] related fields consol_company_id vie the move on move line [IMP] view + groups and some coding [FIX] account domain on company for difference account [FIX] related field type to many2one [FIX] returned generated lines + wrong balance in difference lines [ADD] consolidation group by on move line [ADD] some help tooltips [IMP] coding. (make flake8 happy) [ADD] consolidation analysis view on account.move.line [FIX] label [ADD] more defaults value in wizards and fix forgotten print [FIX] None return in public function + conso. differences smaller than cent will not be generated [FIX] Consolidation difference uses currency is_zero instead of float_is_zero + small style cleanup in method
[ADD] label for user informations + fix mapping account function [ADD] translation + PEP8 [FIX] remove line + context propagation [FIx] display name of the subsidary instead of display the browse record in the error message
Fix currency rate precision in move line label Display consolidated move lines instead of moves Use year and month instead of date from/to Use consolidation profiles in accounting settings for configuration Isolate consolidation demo data from main_company Fix tests and pep8 Generate all consolidation entries in YTD mode instead of period Implement group by account, partner, analytic_account on profile Fix consolidation checks wizard Various fixes, improvements, added comments Try to fix travis Fix computation of all subs accounts Remove groupbys and distinct intercompany partners Raise an error if user is not connected to consolidation company Fix consolidation checks wizard Fix groups and XMLIDs Add record rules and tests Improve README Fix different small things Add more demo accounts Use only one security group instead of two Fix flag reset on move lines, add some logs to find errors more easily Fix display name depends on itself warning Improve description in wizard view Make tests flake8 compliant Cosmetics Ignore travis flake8 error Reverse the consolidation moves on the first day of the month Add check on companies allowed Improve test scenariis Code improvements after review Change TestAccountCompanyRules to SavepointCase Improve consolidation checks view Corrections according to review Fix W503 flake8 error
as wizard setting not properly works with O2M fields replace it on tree view, for additional convenience add consolidation profile into company view
As the consolidated flag is a technical field that is only used during the consolidation process to distinct move lines that are already processed from others, we can safely bypass the ORM to avoid costly overrides in custom modules that are installed.
Currently translated at 98.1% (104 of 106 strings) Translation: account-consolidation-12.0/account-consolidation-12.0-account_consolidation Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-12-0/account-consolidation-12-0-account_consolidation/pt_BR/
Currently translated at 89.6% (95 of 106 strings) Translation: account-consolidation-12.0/account-consolidation-12.0-account_consolidation Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-12-0/account-consolidation-12-0-account_consolidation/es/
Currently translated at 9.4% (10 of 106 strings) Translation: account-consolidation-12.0/account-consolidation-12.0-account_consolidation Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-12-0/account-consolidation-12-0-account_consolidation/hr/
Currently translated at 100.0% (106 of 106 strings) Translation: account-consolidation-12.0/account-consolidation-12.0-account_consolidation Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-12-0/account-consolidation-12-0-account_consolidation/pt_BR/
Currently translated at 100.0% (106 of 106 strings) Translation: account-consolidation-12.0/account-consolidation-12.0-account_consolidation Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-12-0/account-consolidation-12-0-account_consolidation/pt/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-consolidation-14.0/account-consolidation-14.0-account_consolidation_oca Translate-URL: https://translation.odoo-community.org/projects/account-consolidation-14-0/account-consolidation-14-0-account_consolidation_oca/
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For funtcional test @Nantikan23