Skip to content

Conversation

@yagho-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Nov 3, 2025

This PR targets the un-managed branch odoo-dev/odoo:18.0-rd-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.

Commit message it wrongly formatted 👀 You can follow this guideline:
https://www.odoo.com/documentation/17.0/contributing/development/git_guidelines.html#commit-message-structure

<field name="name"/>
<field name="ref"/>
<field name="invoice_date"/>
<field name="balance"/>

Choose a reason for hiding this comment

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

While this works well, another solution might be to use the debit and credit, why ? It will allow the user to type a positive number and have also all the negative since debit and credit are always positive 😄

@yagho-odoo yagho-odoo force-pushed the 18.0-accountingOnboarding-t1-yagho branch from c745284 to 1eb4115 Compare November 5, 2025 10:14
@yagho-odoo yagho-odoo changed the title [ADD] accounting: added a search field to search by balance [IMP] account: added a search field to search by debit/credit Nov 5, 2025
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.

Commit message is still empty, don't forget to add a small description of the change being made and the task-id like said in the guideline here
https://www.odoo.com/documentation/17.0/contributing/development/git_guidelines.html#commit-message-structure

Comment on lines 306 to 309
<field name="name" string="Journal Item" filter_domain="[
'|', '|', '|',
('name', 'ilike', self), ('ref', 'ilike', self), ('account_id', 'ilike', self), ('partner_id', 'ilike', self)]"/>
<field name="name"/>

Choose a reason for hiding this comment

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

Indent here is not necessary ? Keep it like it was 😄

<field name="ref"/>
<field name="invoice_date"/>
<field name="debit" string="Debit/Credit" filter_domain="[
'|',('debit', '=', self), ('credit', '=', self)]"

Choose a reason for hiding this comment

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

you can put everything in one line 😄

@yagho-odoo yagho-odoo force-pushed the 18.0-accountingOnboarding-t1-yagho branch from 1eb4115 to 384188e Compare November 5, 2025 15:58
…t/credit

when typing a number in the search, a new field depending on either the debit or credit can now be used
to filter journal items, this is better than using just the balance field because the balance can be negative
while debit or credit are always positive values
task id: 5231294
@yagho-odoo yagho-odoo force-pushed the 18.0-accountingOnboarding-t1-yagho branch from 384188e to c85e8bb Compare November 5, 2025 15:59
@yagho-odoo yagho-odoo changed the title [IMP] account: added a search field to search by debit/credit [IMP] account: add a search field to search for journal items by debit/credit Nov 5, 2025
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