Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

feat: Allow module writers to override / hook into the SMART style_url #6919

Closed
adunsulag opened this issue Oct 9, 2023 · 0 comments · Fixed by #6922
Closed

feat: Allow module writers to override / hook into the SMART style_url #6919

adunsulag opened this issue Oct 9, 2023 · 0 comments · Fixed by #6922
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Milestone
7.0.2

Comments

@adunsulag
Copy link
Member

SMART apps have the option of mimicking some of the basic styling elements of the OpenEMR configuration. Currently we only support a fixed static url with hard coded styles from the light theme for the SMART style url. We should allow module writers to override this value for module themes / etc.
adunsulag added a commit to DiscoverAndChange/openemr that referenced this issue Oct 10, 2023
@adunsulag
Twigified SMART Style url for module writers
be8c3f6 
Fixes openemr#6919
Moves location of the SMART URL to be part of the OAuth2 url scheme
which lets us customize it and use it in twig.

Fires off a TemplateRender event now that allows module writers to
override the template if they want to.

Changed up the php documentation in the Capability statement with some
minor url fixes.

Removed unused code where we had leftover SMART style code.
@adunsulag adunsulag mentioned this issue Oct 10, 2023
Merged
adunsulag added a commit that referenced this issue Oct 11, 2023
@adunsulag
Twigified SMART Style url for module writers (#6922)
bdbb2bd 
Fixes #6919
Moves location of the SMART URL to be part of the OAuth2 url scheme
which lets us customize it and use it in twig.

Fires off a TemplateRender event now that allows module writers to
override the template if they want to.

Changed up the php documentation in the Capability statement with some
minor url fixes.

Removed unused code where we had leftover SMART style code.
@adunsulag adunsulag changed the title Feat: Allow module writers to override / hook into the SMART style_url feat: Allow module writers to override / hook into the SMART style_url Nov 16, 2023
@adunsulag adunsulag added this to the 7.0.2 milestone Nov 16, 2023
@adunsulag adunsulag added the developers This issue targets an issue that is for developers/collaborators/module writers/technical users label Nov 16, 2023
# for free to join this conversation on GitHub. Already have an account? # 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
7.0.2
Development

Successfully merging a pull request may close this issue.

feat: Twigified SMART Style url for module writers - Fixes #6919 DiscoverAndChange/openemr
1 participant
@adunsulag