We do expect that all trade conditions be attached to a business process, so for manual accounting
   the condition changed.

    Se would like to skip consistency check if the invoice isn't automated (ledger) and has no lines,
   otherwise this consistency check should still apply as usual.

   This patch would replace guard for deliver_action to not call isDivergent that would make slow when open a Transaction (I don't remember the reason exactly, probably due large simulation trees).

   This breaks guards when simulation is not used (so it will never be solved) since the transition would never be allowed to the accountant.

   So if isDivergent proves itself slow, we should patch differently, for now keep the original implementation, since it is proven to work.

    Accounting Manager should be Assignee (not Assignor) to start Accounting Periods.

   Only require project when ledger is automated
   Accept no ledger (1 or 0) on accounting transactions in case manual accounting is used.

   Let the site administrator configure proper accounting instead.

   Business Process are mandatory for Trade Condition, so this process covers
   manual accounting.

   Include a constraint to ensure group is set when function/accounting/{agent,manager} is used on
   assignments.

   This custom approach is used on Panel View and it is only working properly for Panel View since the gadget that will be redirected into is the slapos_panel_view.

   When use normal renderjs_runner it may cause confusion, so keep the URLs just pointing to normal locations.

   This helps administrator to identify which project the ticket is related too.

    This ticket list is broken and it is duplicated with Panel View anyway.

   This information is duplicated with Panel View anyway, no reason to keep it

   Whenever the serialization is JSON-in-xml the parameters are uploaded into {'_' : '{}') format.

    Add mixin to Remote Node and Instance Node, so they can respond to getAccessStatus calls.

    Despite we don't set data on it (same for Slave Instance), it is important to respond this API to display
    Property the Status Gadgets on the API.

    Later on we can setAccessStatus based on Software Instance (for Instance Node) or Compute Node
    based on some indirect criteria, but this not covered by this commit.

This part was not following the new design.
Use the new Type variation document to define how much a instance consumes the compute node capacity.

Manager were using it on previous oss web site

It will reduce the transaction time, and so, reduce conflict cost

Ensure instance is indexed before propagating to the remote node

Do not propagate any url/type change (created by destroy and rename action).

slapos_crm: do not create ticket if we do not have any information for the software installation status

This was the behaviour before the virtual master migration.
Restore it, to not create too many tickets for now.

Paramers are not propagated from Instance Tree to Software Instance, as the manager does not have access to the Open Sale Order AND the customer person.

UID is not a list

The `_verbose` attribute of `SlapOSInstanceTestCase` was unused and it
is now removed.
As the `makeModuleSetUpAndTestCaseClass` needed to be edited to remove
`_verbose` from the class type instantiation, that function has been
formatted, documented and typed.

See merge request nexedi/slapos.core!664