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: add lab test rate to default selling price list #27328

Closed
wants to merge 1 commit into from

Conversation

rtdany10
Copy link
Contributor

@rtdany10 rtdany10 commented Sep 3, 2021

Billable lab test's rates weren't added to the default selling price list in cases where multiple selling price lists were present.
This PR adds the rates to the default one.

@rtdany10 rtdany10 changed the title fix: add lab test rate to default price list fix: add lab test rate to default selling price list Sep 3, 2021
@coveralls
Copy link

Coverage Status

Coverage remained the same at 51.503% when pulling 0848d43 on rtdany10:patch-6 into 798b464 on frappe:develop.

@pateljannat pateljannat self-assigned this Sep 3, 2021
@rtdany10
Copy link
Contributor Author

rtdany10 commented Sep 4, 2021

@pateljannat please add backport labels as well.

@ChillarAnand
Copy link
Contributor

@rtdany10 Thanks for submitting the patch.

We are moving healthcare module into a seperate app and it is available at https://github.com/frappe/healthcare. All new features/fixes will be developed here.

The above patch will be available in v13 hotfix as well. Tracking it at #27353

@rtdany10 rtdany10 deleted the patch-6 branch October 10, 2021 11:53
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants