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

Updating title using React-Helmet #89

Closed
tmbtech opened this issue Aug 3, 2015 · 11 comments
Closed

Updating title using React-Helmet #89

tmbtech opened this issue Aug 3, 2015 · 11 comments

Comments

@tmbtech
Copy link
Contributor

tmbtech commented Aug 3, 2015

@erikras How do you feel about adding React-Helmet to update titles and meta data? This component would allow developers to update the title/meta tags based on the view.

I'm part of the team that open sourced it. I can PR some code, but I wanted to check with you before I started writing code.

lmk, thanks.

@erikras
Copy link
Owner

erikras commented Aug 3, 2015

I've been using react-document-meta in my own project. Aside from your personal pride (😀), are there significant benefits to helmet?

@tmbtech
Copy link
Contributor Author

tmbtech commented Aug 3, 2015

I personally haven't used react-document-meta.... I'll take it back to the team and find out!

@tmbtech
Copy link
Contributor Author

tmbtech commented Aug 3, 2015

It looks like they are pretty similar, the biggest advantage would be hands on support, I can always bubble up any issues to the team. :)

We are using React-Helmet in all of our new react websites, we are also seeking feedback so we can improve on our open source workflow. We plan to release more and the feedback would help tremendously.

@erikras
Copy link
Owner

erikras commented Aug 3, 2015

Good enough reasons for me. PR away!

@korczis
Copy link
Contributor

korczis commented Aug 4, 2015

So what is the plan with this?

@tmbtech
Copy link
Contributor Author

tmbtech commented Aug 4, 2015

I'll issue a PR ASAP.

@tmbtech
Copy link
Contributor Author

tmbtech commented Aug 6, 2015

@erikras I just started integrating react-helmet, there are only a few TODO items left. Feel free to take a look at the code. feature/react-helmet

TODO:

  • Update Html.js jsdocs
  • Update DISABLE_SSR section in server.js
  • Refactor / clean up code.
  • Address any feedback you have.

(I'm doing this outside of work hours, I'll try to finish it up by thursday / friday)

@andreimc
Copy link

andreimc commented Aug 8, 2015

👍

@tmbtech
Copy link
Contributor Author

tmbtech commented Aug 17, 2015

Sorry, I got slammed at work, I'll have this done tonight.

@codepunkt
Copy link
Contributor

@tmbtech Does react-helmet work with SSR?

@dmitry
Copy link

dmitry commented Sep 10, 2015

Looks like react-helmet could be used to render on the server side: https://github.com/nfl/react-helmet#server-usage

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

No branches or pull requests

6 participants