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.

Sign up for GitHub

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

Jump to bottom

fix(POS Profile): replace cur_frm with frm #28390

Merged
merged 2 commits into from
Dec 1, 2021
Merged

fix(POS Profile): replace cur_frm with frm #28390

merged 2 commits into from
Dec 1, 2021

Conversation

rtdany10
Copy link
Contributor

@rtdany10 rtdany10 commented Nov 15, 2021 •
edited

POS Profile:
Replace cur_frm with frm.
Add is_group and company filters to account_for_change_amount field.

#no-docs

@stale stale bot added the inactive label Nov 30, 2021
@rtdany10
Copy link
Contributor Author

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within a week if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@nextchamp-saqib @ankush or any other collaborator✋

@stale stale bot removed the inactive label Nov 30, 2021
@frappe frappe deleted a comment from stale bot Nov 30, 2021
@nextchamp-saqib nextchamp-saqib merged commit 3708cf1 into frappe:develop Dec 1, 2021
mergify bot pushed a commit that referenced this pull request Dec 1, 2021
@rtdany10
fix(POS Profile): replace cur_frm with frm (#28390)
83c7609 
(cherry picked from commit 3708cf1)
@mergify mergify bot mentioned this pull request Dec 1, 2021
Merged
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
@rtdany10
fix(POS Profile): replace cur_frm with frm (frappe#28390)
10f2fb4
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
@rtdany10
fix(POS Profile): replace cur_frm with frm (frappe#28390)
29eaaa3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nextchamp-saqib nextchamp-saqib nextchamp-saqib approved these changes

Assignees

@nemesis189 nemesis189

Labels
backport version-13-hotfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@rtdany10 @nextchamp-saqib @nemesis189