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(sales invoice): get healthcare items from clinical procedure #394

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

tamburro92
Copy link
Contributor

If you create a Clinical Procedure billable ad use a custom Item Code, instead of procedure name same, fetch the healthcare item from Invoice Sales will fail because it won't find the right item code.
To fix you have to retrieve, together the procedure name, the right item code.

Screenshot 2024-02-18 alle 20 51 28

@tamburro92 tamburro92 changed the title Fix: Get Items From Clinical Procedure in Sales Invoice fix: Get Items From Clinical Procedure in Sales Invoice Feb 18, 2024
@tamburro92 tamburro92 changed the title fix: Get Items From Clinical Procedure in Sales Invoice fix(sales invoice): get healthcare items from clinical procedure Feb 18, 2024
@tamburro92
Copy link
Contributor Author

ok now I have read the rule about "commitlint"...
let me know if I have to apply a new pull request.

@akashkrishna619
Copy link
Collaborator

ok now I have read the rule about "commitlint"... let me know if I have to apply a new pull request.

@tamburro92 ithe changes you did in the title need to be done in the commit messages. Also linters issue need to be solved

@tamburro92
Copy link
Contributor Author

ok I'm done

@akashkrishna619 akashkrishna619 changed the base branch from version-15-hotfix to develop February 21, 2024 09:12
@akashkrishna619
Copy link
Collaborator

akashkrishna619 commented Feb 21, 2024

changing target to base branch to develop as this will be appicable to v14 as well

@akashkrishna619
Copy link
Collaborator

@tamburro92 can you test this with other invoicing as well? (Service Request/ Medication Request / Encounter etc..)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants