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 not mandatory for Sales Invoice #26853

Merged
merged 6 commits into from
Aug 10, 2021
Merged

fix: Pos Profile not mandatory for Sales Invoice #26853

merged 6 commits into from
Aug 10, 2021

Conversation

AfshanKhan
Copy link
Contributor

For Sales Invoice Pos Profile can be set empty.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 42.712% when pulling 4a2303f on AfshanKhan:SI-pos-profile-ignore into 7ba8c82 on frappe:develop.

@nextchamp-saqib nextchamp-saqib changed the base branch from develop to version-13-hotfix August 10, 2021 06:52
@nextchamp-saqib nextchamp-saqib changed the base branch from version-13-hotfix to develop August 10, 2021 06:52
@nextchamp-saqib nextchamp-saqib merged commit af2f527 into frappe:develop Aug 10, 2021
frappe-pr-bot pushed a commit to frappe-pr-bot/erpnext that referenced this pull request Aug 10, 2021
@AfshanKhan @frappe-bot
fix: pos profile not mandatory for Sales Invoice (frappe#26853)
b22231a 
(cherry picked from commit af2f527)
@frappe-pr-bot frappe-pr-bot mentioned this pull request Aug 10, 2021
Merged
This was referenced Aug 10, 2021
Merged
Merged
@AfshanKhan AfshanKhan mentioned this pull request Aug 17, 2021
Merged
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
@AfshanKhan
fix: pos profile not mandatory for Sales Invoice (frappe#26853)
7a18fb9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nextchamp-saqib nextchamp-saqib Awaiting requested review from nextchamp-saqib

Assignees
No one assigned
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
@AfshanKhan @coveralls @nextchamp-saqib