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

Add EM functionality to report pages #1

Open
ThomasVermes opened this issue Jan 4, 2023 · 5 comments
Open

Add EM functionality to report pages #1

ThomasVermes opened this issue Jan 4, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@ThomasVermes
Copy link

Dear Adam, once again great EM allowing a bettere redcap usability!!
if I understood correctly, it is possible to open the modal only in a redcap_data_entry_form right?
would be possible to active the function also in other pages?
We would like to use it in report

image

nb: to make the report field work as "link" we used this little trick saving the field as calctext

@CALCTEXT(concat('<a class="modale"','>Open Modal<','/a>'))

@aanunez aanunez changed the title Add the functionality also elswhere Add EM functionality to report pages Jan 4, 2023
@aanunez
Copy link
Member

aanunez commented Jan 4, 2023

Hey @ThomasVermes. I'm not sure if I'll be able to add this functionality, but I'll leave this open with a few notes on it.

Possible issues:

  • We would need to collect report id so we know when to load settings
  • Collect a new "origin" setting with default value on data form and another option "report"
  • URLs used on reports would need to have record id populated in JS so we could build them when we have the report ready

@aanunez aanunez added the enhancement New feature or request label Jan 4, 2023
@ThomasVermes
Copy link
Author

ah, then is more complicate than what I tought.
I thought it might be an option to save the link in a variable and just opening the link with the module class..

@aanunez aanunez added the wontfix This will not be worked on label May 10, 2024
@aanunez aanunez self-assigned this May 10, 2024
@aanunez
Copy link
Member

aanunez commented May 10, 2024

I've decided that I won't be implementing this. I think wanting to open a record as a modal from a report won't see much use compared to just opening in a new tab. In addition to that the issues above also exist. I do think this would be easier to add to the Report Tweaks EM, but due to not see the use case I've decided to not add it as an issue there either.

@aanunez aanunez closed this as completed May 10, 2024
@ThomasVermes
Copy link
Author

Dear Adam,
not to reopen this issue ;)
but a case would be that user have a list of patient and they need to entry some dates from an external tool (which is difficult to import). With the Modal the user would be redirected not to only to the record, but also to the right instrument.
Here below a small gif as example of what I've implemented (with javascript and more "help"-Variables).
potential_use_of_portal_em

My users where quite happy with that. Of course they spare probably 2 clicks, but also the fact that they stay on the report page is helpfull..("open link in a new Tab" is not known by everyone =)

That's all, I just wanted to show you some usability =)
cheers
Thomas

@aanunez
Copy link
Member

aanunez commented May 30, 2024

Opening the issue again.

For config we can just have a field drop down per modal config group. If that field is on the report add a button to the cell that opens to the defined modal destination.

@aanunez aanunez reopened this May 30, 2024
@aanunez aanunez removed the wontfix This will not be worked on label Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants