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

bug: Immunization Date of VIS Statement Help Link Broken #7844

Open
adunsulag opened this issue Nov 26, 2024 · 10 comments
Open

bug: Immunization Date of VIS Statement Help Link Broken #7844

adunsulag opened this issue Nov 26, 2024 · 10 comments
Assignees
Milestone

Comments

@adunsulag
Copy link
Member

The current link on the Immunization Date of VIS Statement for US CDC goes here which cannot be found:
https://www.cdc.gov/vaccines/pubs/vis/default.htm

image

I believe it should be set to be here:
https://www.cdc.gov/vaccines/hcp/vis/current-vis.html

Note this is region/language specific so we really need to make this link configurable at the database level for localization efforts.

All versions of OpenEMR that have the VIS Statement link, verified against
Latest OpenEMR Master version (7.0.2 patch 3)

Steps to reproduce.

  1. Attempt to add a new immunization
  2. Click on the help icon question mark
  3. See page not found link.
@JulieThurmes
Copy link
Contributor

I would like to work on this. Would it be possible to clarify the following statement please, "Note this is region/language specific so we really need to make this link configurable at the database level for localization efforts"?

Are you wanting for the user's location and language preferences to be shared with the CDC's site in the link itself? I haven't found that that has produced any changes to the VIS content. Otherwise, I can configure the links to the translated VIS forms on immunize.org and house them in a database. Or maybe you would like to configure the link to the user's region, such as a link to Canada's version of the CDC VIS information?

I appreciate any clarification you can provide, thank you.

@adunsulag
Copy link
Member Author

@JulieThurmes I've been out with the thanksgiving break so my apologies on the delay. My preference would be to have a setting in our globals database that has the link for the user's region. That assumes that every region would have a website to get the VIS for all the vaccines dictated by their jurisdiction.

If we make it a translatable string we'd have the problem that someone's VIS in Spain could be different for the spanish language versus someone's VIS here in the USA for the spanish language. That's what I'm thinking here, but open to suggestions.

The PR you've pushed will fix the bug, but doesn't address the regional nuances that we'll need to resolve eventually. It'd be great if you were up for taking on that piece, or we can save that for someone else and leave this issue open.

@adunsulag adunsulag added this to the 7.0.2.4 milestone Dec 3, 2024
@JulieThurmes
Copy link
Contributor

Thank you so much for the clarification, I would like to continue to work on this.

@adunsulag
Copy link
Member Author

@JulieThurmes do you want to do that as a separate PR, or work within your original PR?

@JulieThurmes
Copy link
Contributor

I apologize, I think I may still be misunderstanding. The requirement to provide a VIS sheet to a patient when administering a vaccine is a federal one in the US. From my understanding and research, there are not equivalent sites for VIS sheets for other regions such as Spain, Canada, etc. since the sheets are unique to the US/CDC.

The publication dates for all the available US VIS sheets are already included in the CDC link: https://www.cdc.gov/vaccines/hcp/vis/current-vis.html. Another option would be to link to the Immunize.org site: https://www.immunize.org/vaccines/vis/about-vis/ where a PDF can be found with a list of all the vaccines and the VIS dates along with translations for the VIS sheets if needed. The second option is a bit more cumbersome for the user in my opinion. Just to note, the dates for the translations of the VIS sheets appear to be the same as the English version.

Again, I do apologize if I am completely misunderstanding the task here. It may be best for me to step aside on this one.

@Mamatha1718
Copy link

Hi adunsulag,
To address the outdated VIS link issue, I’ve made a few updates:

Updated the VIS link:
I replaced the old CDC VIS link with the correct one:
https://www.cdc.gov/vaccines/hcp/vis/current-vis.html.
This ensures users are directed to the right page when they click the help icon.

Proposed a solution for regional links:
To handle region-specific VIS links, I suggest adding a configuration like this in the globals file
This would make the system flexible for localization.

Tested the changes:
I verified that the updated link works correctly when clicking the help icon next to the "Date of VIS Statement" label.

If the team agrees with the regional configuration approach, I’d love to help implement it further. Please assign this issue to me so I can continue working on it. Let me know if there’s anything else you’d like me to address!

Thanks! 😊

@adunsulag
Copy link
Member Author

@JulieThurmes I thought some of the countries in the EU had their own VIS sheets as you're right the VIS statement date recording is a US federal requirement. I'll ping some of our EU members and see if there's something different here. Otherwise let's go ahead and bring your PR in.

I do know not everyone follows the same immunization schedule as the US which is what I've had in my head, however, the VIS sheet is for a specific immunization, and not the entire immunization schedule so I may have been taking this too far, I'll find out more info and can let you know.

@adunsulag
Copy link
Member Author

@Mamatha1718 I'd prefer you collaborate with @JulieThurmes if you're not already. I don't see a reason in having redundant work done here and @JulieThurmes was already working on this issue. If you're looking for some other work that first time contributors can do I can send some your way.

@Mamatha1718
Copy link

Thank you for the clarification! Please let me know if there’s anything else I can do to help with this issue or any updates regarding regional differences. Happy to assist further! 😊

@Mamatha1718
Copy link

Thank you for pointing that out! I’ll let @JulieThurmes continue with this issue. I’d appreciate it if you could send some other first-time contributor tasks my way. Looking forward to contributing!

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

No branches or pull requests

3 participants