* feat: subcontracting inward
* feat: stock reservation
* feat: subcontracting delivery
* feat: all remaining stuff
* fix: linter errors
* fix: patch
* fix: modify stock entry type validation
* fix: customer provided item cost field mandatory validation
* fix: failing tests
* fix: failing tests
* fix: subcontracting controlller
* refactor: semi final
* refactor: final
* chore: resolve conflicts
* refactor: changes requested
* fix: reservation transfer of extra qty
* fix: consider add cost for customer provided rate field
* test: create test data
* test: subcontracted sales order (partial)
* test: fin
* fix: do not add self RM in DN created from SI
* fix: failing test case
* fix: conflicting function name
* refactor: final changes
* fix: more bugs
* perf: various and major performance improvements
* fix: consider warehouse as well in all queries
* fix: same item code with diff warehouse in manufacture entry
* refactor: readability
* fix: frontend validations
* perf: replace query inside loop with single query
* fix: set additional item flag to true when extra customer provided item is received
* fix: bugs found by coderabbit
* fix: more coderabbit bugs
* fix: add validation to disallow cancellation of manufacturing entry
* perf: use cached values wherever it makes sense
* test: fix redundant insert to child tables
* fix: consider SI return of billed self RM
* fix: bug found by coderabbit
---------
Co-authored-by: Mihir Kandoi <mihirkandoi@Mihirs-MacBook-Air.local>
* perf(get_item_details): use cached doc, avoid full get_doc
* perf: short-circuit common usecase of same UOM
* perf: use cached doc for inspection
* perf: use cached valuation method and other item props
It rarely if ever changes after initial configuration
* feat: change sabb qty automatically incase of internal transfer PR if sabb only has 1 batch
* fix: prevent creation of SABB on every save
* perf: optimize code
* fix: remove unnecessary conditon
* refactor: change if to elif
* fix: remove dn_item_qty and set to item.qty
* test: added test
Some queries still use "timestamp" function instead of "posting_datetime". In my instance single stock entry cancel ends with request timeout. Using "posting_datetime" field directly improves the situation.
cont: https://github.com/frappe/erpnext/pull/46293
