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: remove stock balance report limit #26773

Merged
merged 1 commit into from
Aug 2, 2021

Conversation

ankush
Copy link
Member

@ankush ankush commented Aug 2, 2021

This validation existed when the "prepared report" functionality didn't exist. This isn't required right now and reports often generate in a matter of seconds with 1M SLE.

@ankush ankush changed the title fix: increase (4x) stock balance report limit fix: remove stock balance report limit Aug 2, 2021
@ankush ankush removed the request for review from marination August 2, 2021 14:07
@ankush ankush merged commit b3740e9 into frappe:develop Aug 2, 2021
frappe-pr-bot pushed a commit to frappe-pr-bot/erpnext that referenced this pull request Aug 2, 2021
ankush added a commit that referenced this pull request Aug 3, 2021
(cherry picked from commit b3740e9)

Co-authored-by: Ankush <ankush@iwebnotes.com>
frappe-pr-bot pushed a commit to frappe-pr-bot/erpnext that referenced this pull request Aug 13, 2021
ankush added a commit that referenced this pull request Aug 13, 2021
(cherry picked from commit b3740e9)

Co-authored-by: Ankush <ankush@iwebnotes.com>
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
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