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] Orders not creating from the POS for new customer if customer name is based on naming series #8933

Merged

Conversation

rohitwaghchaure
Copy link
Collaborator

@rohitwaghchaure rohitwaghchaure commented May 19, 2017

Issue
Customer name by naming series
screen shot 2017-05-19 at 2 38 07 pm

bbb6e51a-3c81-11e7-94cd-c80041a2e5ea

Fixed #8923
Fixed #8621
Fixed #9110

@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 044dd6c to 6e246bd Compare May 19, 2017 09:54
@nabinhait nabinhait requested a review from mbauskar May 24, 2017 12:11
@nabinhait
Copy link
Member

@mbauskar Can you test this?

Copy link
Contributor

@mbauskar mbauskar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while syncing Sales Invoice the same customer is synced multiple times.

@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 6e246bd to 4c6be49 Compare May 31, 2017 09:55
@rohitwaghchaure
Copy link
Collaborator Author

rohitwaghchaure commented May 31, 2017

@mbauskar Fixed!

@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch 6 times, most recently from c57091d to 783c3a5 Compare May 31, 2017 12:55
@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 783c3a5 to 2503f5e Compare June 1, 2017 06:25
"in_standard_filter": 0,
"label": "Customer POS ID",
"length": 0,
"no_copy": 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no_copy=1

"no_copy": 0,
"permlevel": 0,
"precision": "",
"print_hide": 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

print_hide=1

"columns": 0,
"fieldname": "customer_pos_id",
"fieldtype": "Data",
"hidden": 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hidden=1

@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 2503f5e to 588f5a2 Compare June 2, 2017 12:42
@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 8d50722 to 89456d1 Compare June 5, 2017 06:36
@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch 3 times, most recently from 7b3a7c0 to 697d67c Compare June 5, 2017 10:09
@rohitwaghchaure rohitwaghchaure force-pushed the pos_customer_issue_for_naming_series branch from 697d67c to 449e443 Compare June 5, 2017 11:08
@nabinhait nabinhait merged commit d2be55b into frappe:hotfix Jun 7, 2017
tundebabzy pushed a commit to tundebabzy/erpnext that referenced this pull request Jun 13, 2017
…ame is based on naming series (frappe#8933)

* [Fix] Orders not creating from the POS for new customer if customer name is based on naming series

* [fix] Disabled price field in POS, if Allow User to Edit Rate field is disabled in POS profile

* added customer name in the search bar of the customer

* search customer by phone number and email id in the POS
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

3 participants