Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(patch): sle.serial_no = "\n" causes incorrect queue #29306

Merged
merged 1 commit into from
Jan 16, 2022

Conversation

ankush
Copy link
Member

@ankush ankush commented Jan 16, 2022

extending the previous patch to clean up "\n" in serial_no field.

@ankush ankush added the Skip Manual Testing The changes in this PR does not require manual testing label Jan 16, 2022
@codecov
Copy link

codecov bot commented Jan 16, 2022

Codecov Report

Merging #29306 (d0b522b) into develop (b0cf619) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop   #29306      +/-   ##
===========================================
- Coverage    57.95%   57.89%   -0.07%     
===========================================
  Files         1110     1110              
  Lines        68088    68089       +1     
===========================================
- Hits         39463    39419      -44     
- Misses       28625    28670      +45     
Impacted Files Coverage Δ
erpnext/stock/stock_ledger.py 85.45% <100.00%> (-0.35%) ⬇️
...wise_balance_history/batch_wise_balance_history.py 68.96% <0.00%> (-24.14%) ⬇️
...xt/stock/report/stock_analytics/stock_analytics.py 80.19% <0.00%> (-12.88%) ⬇️
...em_wise_sales_register/item_wise_sales_register.py 51.81% <0.00%> (-9.55%) ⬇️
...unts/report/purchase_register/purchase_register.py 73.28% <0.00%> (-6.85%) ⬇️
...ctype/woocommerce_settings/woocommerce_settings.py 80.76% <0.00%> (-3.85%) ⬇️
...eport/item_variant_details/item_variant_details.py 84.33% <0.00%> (-3.62%) ⬇️
...e/asset_value_adjustment/asset_value_adjustment.py 86.04% <0.00%> (-3.49%) ⬇️
erpnext/education/doctype/student/student.py 73.68% <0.00%> (-3.16%) ⬇️
...e_purchase_register/item_wise_purchase_register.py 75.00% <0.00%> (-3.00%) ⬇️
... and 23 more

@ankush ankush merged commit 66bf21f into frappe:develop Jan 16, 2022
mergify bot pushed a commit that referenced this pull request Jan 16, 2022
This happens due to old data.

(cherry picked from commit 66bf21f)
ankush added a commit that referenced this pull request Jan 16, 2022
)

This happens due to old data.

(cherry picked from commit 66bf21f)

Co-authored-by: Ankush Menat <ankush@frappe.io>
@ankush ankush removed the Skip Manual Testing The changes in this PR does not require manual testing label Mar 9, 2022
KrithiRamani pushed a commit to KrithiRamani/erpnext that referenced this pull request Mar 14, 2022
frappe#29309)

This happens due to old data.

(cherry picked from commit 66bf21f)

Co-authored-by: Ankush Menat <ankush@frappe.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant