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: pos invoices consolidation case with permlevel #29011

Merged
merged 2 commits into from
Jan 10, 2022

Conversation

nextchamp-saqib
Copy link
Member

The field ignore_pricing_rule had permlevel = 1 by default
If some users have perm level zero and are expected to create pos closing entries, they faced problems while consolidating.

POS Invoices with pricing rules were causing problems while getting consolidated as ignore_pricing_rule cannot be set by users having perm level zero.

@nextchamp-saqib nextchamp-saqib merged commit e5b649b into frappe:develop Jan 10, 2022
mergify bot pushed a commit that referenced this pull request Jan 10, 2022
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