Skip to content

Conversation

@JasicMila
Copy link

New spanish assets model created and related to accounts es_common and es_full

task - 5867581

@robodoo
Copy link

robodoo commented Jan 29, 2026

This PR targets the un-managed branch odoo-dev/odoo:19.0-l10n_es-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

Nice one, just two comments ! 😄

"asset_motor_vehicles","a352000","281800","681000","Motor vehicles","4","12"
"asset_other_property_plant_equipment","a352100","281900","681000","Other property, plant and equipment","5","12"
"asset_investments_in_buildings","a352200","282000","682000","BInvestments in buildings","30","12"

Choose a reason for hiding this comment

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

Extra line for no reason 👀

"asset_development","a322000","280000","680000","Development ","5","12"
"asset_administrative_concessions","a323000","280000","682000","Administrative concessions","5","12"
"asset_industrial_property","a330000","280000","682000","Industrial property","10","12"
"asset_goodwill","a340000","280000","682000","Goodwill","10","12"

Choose a reason for hiding this comment

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

Goodwill should be in another account.asset file 👀

Copy link
Author

Choose a reason for hiding this comment

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

acknowledged!

Choose a reason for hiding this comment

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

you can remove this one then 😄 Don't forget to remove it also from the common account.account

@JasicMila JasicMila force-pushed the 19.0-l10n_es-new-assets-model-jmil branch from ba2093d to d64c964 Compare January 29, 2026 14:01
@JasicMila JasicMila requested a review from malb-odoo January 29, 2026 14:02
Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

Few things 😄

@@ -0,0 +1,19 @@
"id","account_asset_id","account_depreciation_id","account_depreciation_expense_id","name","method_number","method_period"
"asset_research","a315000","280000","680000","Research","5","12"

Choose a reason for hiding this comment

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

for account_depreciation_id and account_depreciation_expense_id you added the number, but it should be the id of the account 👀

"asset_development","a322000","280000","680000","Development ","5","12"
"asset_administrative_concessions","a323000","280000","682000","Administrative concessions","5","12"
"asset_industrial_property","a330000","280000","682000","Industrial property","10","12"
"asset_goodwill","a340000","280000","682000","Goodwill","10","12"

Choose a reason for hiding this comment

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

you can remove this one then 😄 Don't forget to remove it also from the common account.account

New assets model created and related to accounts es_common and es_full
task - 5867581
@JasicMila JasicMila force-pushed the 19.0-l10n_es-new-assets-model-jmil branch from d64c964 to 504a8b1 Compare January 29, 2026 15:53
@JasicMila JasicMila requested a review from malb-odoo January 29, 2026 15:54
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.

4 participants