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-profile): Cleanup Form for POS Profile #16208

Merged
merged 1 commit into from Dec 14, 2018

Conversation

Projects
None yet
2 participants
@netchampfaris
Copy link
Contributor

commented Dec 13, 2018

  • Remove pos_profile_name field and use name as Prompt
  • Reorganize form and label sections
  • Remove apply_discount field which was not used

gadgets_8000_desk 1

fix(pos-profile): Cleanup Form for POS Profile
- Remove pos_profile_name field and use `name` as Prompt
- Reorganize form and label sections
- Remove `apply_discount` which was not used

@rmehta rmehta merged commit 4994d84 into frappe:staging-fixes Dec 14, 2018

2 of 4 checks passed

coverage/coveralls Coverage decreased (-0.0008%) to 51.397%
Details
Codacy/PR Quality Review Hang in there, Codacy is reviewing your Pull request.
Details
Travis CI - Pull Request Build Passed
Details
security/snyk - requirements.txt (frappe) No manifest changes detected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.