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

Opened visor v5 #506

Open
wants to merge 24 commits into
base: release/5.2.0
Choose a base branch
from
Open

Opened visor v5 #506

wants to merge 24 commits into from

Conversation

kokushkin
Copy link

@kokushkin kokushkin commented Feb 3, 2020

Injects arbitrary html strings through the HelmetsOpenedVisor element.
This closes #323 . See https://www.npmjs.com/package/react-helmet-with-visor . We had to make the fork and the package for our needs, but would be happy to merge back to your project.

@jeffyamada
Copy link

This looks fantastic. I wish it were merged, but we need this badly so I'm giving the fork out a try for now (though like anyone just concerned about longer term maintenance of the package without a formal merge). Thanks @kokushkin !!!

@theskillwithin
Copy link

any news on this?

@kokushkin
Copy link
Author

any news on this?
Hi @theskillwithin , what do you mean news?) I thought the owners of the library could help me out with the tests failures (local tests were ok). But seems like they don't like the idea, isn't it @cwelch5 ? So, there is fork https://www.npmjs.com/package/react-helmet-with-visor what we're using and you might as well. And we can merge everything from here if you are missing something. BTW, I made a video https://www.youtube.com/watch?v=DWPP16LGNOA about the subject. Don't hesitate to reach me out, maybe we can hack something together 😉

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.

3 participants