Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 141
    • Merge requests 141
  • 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
  • nexedi
  • erp5erp5
  • Merge requests
  • !156

Open
Created Aug 17, 2016 by Julien Muchembled@jmOwner
  • Report abuse
Report abuse

amount_generator: refresh "Trade Model" view on deliveries and delivery lines

  • Overview 4
  • Commits 1
  • Changes 29

It's been a few years since getAggregatedAmountList should be called on the delivery or the movement, and a single form for both levels is enough.

This commit also adds a form to view non-aggregated amounts. We still have to decide where to put a link to view it: a new tab would be too much. For this form, it would be also interesting to add a column to the delivery/movement being processed (currently, a private _base attribute on each amount).

Rows don't link anywhere. This could be changed to jump the causality object (e.g. Trade Model Line).

At last, maybe erp5_base or erp5_simulation would be a better place than erp5_trade for these forms, except that it's not trivial to move because they use Base_viewTradeFieldLibrary.

TODO:

  • in contribution/application columns, display variations

/cc @jerome @seb @kazuhiko @yusei

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: view-aggregated-amounts
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7