feat: new column 'Time taken to Deliver' in sales order analysis

[ruthra-kumar]

Current Report

'Sales Order Analysis' helps in quickly identifying billed amount, pending amount, delay in days and more information. Along with this a new column is added

New Column - 'Time taken to Deliver'

This denotes the no of days taken to completely deliver an item. This helps in identifying bottlenecks and get more efficient.

Image

no-docs

[codecov]

Codecov Report

Merging #28842 (8aa9481) into develop (92e86f2) will decrease coverage by 0.05%.
The diff coverage is n/a.

❗ Current head 8aa9481 differs from pull request most recent head da67403. Consider uploading reports for the commit da67403 to get more accurate results

@@             Coverage Diff             @@
##           develop   #28842      +/-   ##
===========================================
- Coverage    55.60%   55.54%   -0.06%     
===========================================
  Files         1128     1125       -3     
  Lines        67287    66984     -303     
===========================================
- Hits         37412    37205     -207     
+ Misses       29875    29779      -96     

Impacted Files	Coverage Δ
...eport/sales_order_analysis/sales_order_analysis.py	0.00% <ø> (ø)
...wise_balance_history/batch_wise_balance_history.py	68.96% <0.00%> (-22.42%)	⬇️
...ctype/woocommerce_settings/woocommerce_settings.py	80.00% <0.00%> (-4.00%)	⬇️
...value/warehouse_wise_item_balance_age_and_value.py	92.40% <0.00%> (-2.54%)	⬇️
erpnext/stock/reorder_item.py	76.06% <0.00%> (-0.86%)	⬇️
...ice_creation_tool/opening_invoice_creation_tool.py	60.83% <0.00%> (-0.81%)	⬇️
erpnext/selling/doctype/sales_order/sales_order.py	77.20% <0.00%> (-0.77%)	⬇️
erpnext/crm/doctype/lead/lead.py	64.46% <0.00%> (-0.71%)	⬇️
erpnext/accounts/deferred_revenue.py	75.43% <0.00%> (-0.44%)	⬇️
erpnext/stock/stock_ledger.py	85.21% <0.00%> (-0.37%)	⬇️
... and 23 more

[marination]

@nextchamp-saqib @ruthra-kumar This report was introduced via #22055 to replace "Ordered Items to be Billed" and "Ordered Items to be Delivered and Billed"
The right join to DN makes it only fetch those transaction that have DNs against them, it does not fetch SOs that are pending to Deliver and Bill
It could use a test too, given that this could have been identified by CI

Also this column is a bit confusing with the Delay column, this needs documentation too https://docs.erpnext.com/docs/v13/user/manual/en/selling/sales-analytics

[ruthra-kumar]

@marination thanks for reporting this. Will create a fix.