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

feat: Prescriptions Dialog move to twig to allow module writers to extend / modify the view #7496

Closed
adunsulag opened this issue Jun 19, 2024 · 0 comments · Fixed by #7495
Closed

feat: Prescriptions Dialog move to twig to allow module writers to extend / modify the view #7496

adunsulag opened this issue Jun 19, 2024 · 0 comments · Fixed by #7495
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users

Comments

@adunsulag
Copy link
Sponsor Member

As a developer I want to be able to change up the view presentation, hide fields, change up default options, etc for the prescription edit screen. We should change from the old smarty template style to the twig format so I can provide an override template in my module to customize the look of the screen without modifying the core code.
@adunsulag adunsulag added the developers This issue targets an issue that is for developers/collaborators/module writers/technical users label Jun 19, 2024
adunsulag added a commit to DiscoverAndChange/openemr that referenced this issue Jun 19, 2024
@adunsulag
Fixes openemr#7496 move erx general edit to twig
72700e5 
Made it so the edit prescription dialog is in twig instead of in the old
html format.  This fixes openemr#7496.  I've done some testing but probably
need to do more testing to verify we've handled all the use cases.
@adunsulag adunsulag mentioned this issue Jun 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feat openemr fix 7480 7494 email prescription DiscoverAndChange/openemr
1 participant
@adunsulag