Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1713

Improve bank accounts selection on accounting transactions

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Jérome Perrin requested to merge fix/bank_accounts into master Dec 21, 2022
  • Overview 1
  • Commits 7
  • Pipelines 5
  • Changes 33

Fix small problems displaying bank accounts on accounting transactions and be more strict during validation:

  • bank account must be validated
  • bank account must belong to the section

The latter was a problem for scenarios like this:

  • create transaction for supplier 1 with supplier 1 bank account
  • clone transaction
  • change to supplier 2 the bank account remains as supplier 1 bank account, although it became an invoice from supplier 2. The transaction could be validated as long as the bank account is not invalidated
Edited Dec 21, 2022 by Jérome Perrin
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/bank_accounts
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7