feat(buying): quotation number in supplier quotation

[hrwX]

• Add Quotation Number in Supplier Quotation similar to Order Confirmation Number in Purchase Order.

Docs: https://docs.erpnext.com/docs/v13/user/manual/en/buying/supplier-quotation#32-more

[barredterra]

@Mergifyio backport version-13-hotfix

[mergify]

backport version-13-hotfix

🟠 Waiting for conditions to match

• merged [:pushpin: backport requirement]

[barredterra]

@hrwX please mention the new fields in https://docs.erpnext.com/docs/v13/user/manual/en/buying/supplier-quotation

[hrwX]

@hrwX please mention the new fields in https://docs.erpnext.com/docs/v13/user/manual/en/buying/supplier-quotation

Updated https://docs.erpnext.com/docs/v13/user/manual/en/buying/supplier-quotation#32-more

[sagarvora]

@barredterra @hrwX

I actually have a late request: as per convention, perhaps the number field should come before the date. Also worth noting that similar fields in Purchase Invoice and Sales Invoice have their own section with a column break between Number and Date.

[barredterra]

Thanks, @sagarvora. @hrwX please change accordingly.

[hrwX]

@sagarvora @barredterra Updated the fields order.

[barredterra]

@hrwX what are your thoughts about the separate section?

[hrwX]

@hrwX what are your thoughts about the separate section?

• Even if we see in Purchase Order, Order Confirmation and Date do not have their own sections, either we add a new section for both or we keep the fields in the first section of the form itself where its clearly visible.

[codecov]

Codecov Report

Merging #28420 (f136755) into develop (fcb614d) will decrease coverage by 0.03%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##           develop   #28420      +/-   ##
===========================================
- Coverage    55.24%   55.21%   -0.04%     
===========================================
  Files         1122     1122              
  Lines        66828    66828              
===========================================
- Hits         36922    36900      -22     
- Misses       29906    29928      +22     

Impacted Files	Coverage Δ
...eport/item_variant_details/item_variant_details.py	31.32% <0.00%> (-53.02%)	⬇️
...work_order_stock_report/work_order_stock_report.py	50.00% <0.00%> (-50.00%)	⬇️
...value/warehouse_wise_item_balance_age_and_value.py	92.40% <0.00%> (-2.54%)	⬇️
erpnext/stock/report/stock_ledger/stock_ledger.py	74.79% <0.00%> (-2.44%)	⬇️
erpnext/stock/report/stock_ageing/stock_ageing.py	89.05% <0.00%> (-1.46%)	⬇️
.../report/stock_projected_qty/stock_projected_qty.py	86.84% <0.00%> (-1.32%)	⬇️
...eport/production_analytics/production_analytics.py	69.51% <0.00%> (-1.22%)	⬇️
erpnext/stock/stock_ledger.py	85.73% <0.00%> (-1.22%)	⬇️
erpnext/support/doctype/issue/issue.py	63.41% <0.00%> (-0.98%)	⬇️
erpnext/accounts/deferred_revenue.py	75.43% <0.00%> (-0.88%)	⬇️
... and 17 more

[sagarvora]

I think people will get confused between transaction_date and quotation_date. Any reason why these should differ at all?

[sagarvora]

@hrwX Can you please update the docs accordingly?

[mergify]

backport version-13-hotfix

✅ Backports have been created

• #28827 feat(buying): quotation number in supplier quotation (backport #28420) has been created for branch version-13-hotfix

[barredterra]

I updated the docs