Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 10, 2025

Before Change

The system used to run a query to get the current value for each quantity. So, if the quantity was 200, the system would run 200 SQL queries.

After Change

Now, the system will run only two queries. The first query retrieves the current value of the prefix, and the second query updates the current value of the prefix in the tabSeries table after creation of all serial nos.


This is an automatic backport of pull request #50436 done by Mergify.

(cherry picked from commit 19a9497)

# Conflicts:
#	erpnext/stock/serial_batch_bundle.py
@mergify
Copy link
Contributor Author

mergify bot commented Nov 10, 2025

Cherry-pick of 19a9497 has failed:

On branch mergify/bp/version-15-hotfix/pr-50436
Your branch is up to date with 'origin/version-15-hotfix'.

You are currently cherry-picking commit 19a9497273.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   erpnext/stock/serial_batch_bundle.py

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@github-actions github-actions bot added needs-tests This PR needs automated unit-tests. stock labels Nov 10, 2025
Removed unused voucher_no and posting_date retrieval.
@rohitwaghchaure rohitwaghchaure removed needs-tests This PR needs automated unit-tests. conflicts labels Nov 11, 2025
@rohitwaghchaure rohitwaghchaure merged commit de14c08 into version-15-hotfix Nov 11, 2025
13 checks passed
@rohitwaghchaure rohitwaghchaure deleted the mergify/bp/version-15-hotfix/pr-50436 branch November 11, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants