search

OCA / account-reconcile

Odoo account reconciliation modules (statements, data completion...)
https://odoo-community.org/psc-teams/banking-10
GNU Affero General Public License v3.0
127 stars 369 forks source link

[16.0][IMP] account_statement_base: add form view for account_bank_statement #647

Closed SergioBustamanteFL closed 1 month ago

SergioBustamanteFL commented 2 months ago

Added a form view for account_bank_statement model

OCA-git-bot commented 2 months ago

Hi @alexis-via, some modules you are maintaining are being modified, check this out!

OCA-git-bot commented 2 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

SergioBustamanteFL commented 1 month ago

@pedrobaeza is this OK for merge ?

sergio-teruel commented 1 month ago

Hi @SergioBustamanteFL

I think that this view show fields not needed.. the view is not clear...

I have opened a PR (I didn't know of this) with a more simple view like this.

With correct display warning, smart button to access to lines directly...

image

Also... if you add a line you have an error due to Odoo error. I will open a PR to Odoo...

If you want to improve your eyesight, perfect, if not we can continue in my PR.

sergio-teruel commented 1 month ago

PR to odoo to fix the creation of statement lines inside the statement form view https://github.com/odoo/odoo/pull/167224

SergioBustamanteFL commented 1 month ago

@sergio-teruel I like your approach. Let's continue in your PR. Thank you!