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

fix: dont add when helmetdata already has same instance #176

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

king-of-emotion
Copy link

I use react-helmet-async like https://github.com/staylor/react-helmet-async#usage-without-context example, but every time I access the website that has Helmet Component, helmetData add same instance and not remove.

Dispatcher class has componentWillUnmount, but it is not called probably because this class render method return null.

So, I want to check the helmetData when add the instance that helmetData already has or not.

sorry for my poor english.

@king-of-emotion king-of-emotion force-pushed the not-add-when-already-has-same-instance branch from 7a72ceb to 82c6bbb Compare July 11, 2022 09:35
@king-of-emotion king-of-emotion changed the title fix: dont add when helmetdata has already same instance fix: dont add when helmetdata already has same instance Jul 11, 2022
@king-of-emotion
Copy link
Author

@staylor please check this pull request.

@gilluminate gilluminate mentioned this pull request Jan 8, 2024
@gilluminate
Copy link

This will also address issues brought up in #213

NextJS apps will be able to use this package after this PR has been merged. Please bump this in priority.

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.

2 participants