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.

Sign up for GitHub

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

Jump to bottom

Remove unncessary title and meta tags #1655

Closed
chrispinkney opened this issue Feb 5, 2021 · 0 comments · Fixed by #1774
Closed

Remove unncessary title and meta tags #1655

chrispinkney opened this issue Feb 5, 2021 · 0 comments · Fixed by #1774
Assignees
@chrispinkney
Labels
5 min fix Fixable in 5 minutes or less area: front-end area: nextjs Nextjs related issues type: enhancement New feature or request
Milestone
1.7 Release

Comments

@chrispinkney
Copy link
Contributor

What would you like to be added removed:
There's a few places or two where we are:

  • Using <title> tags
  • Using <meta> tags
  • Using both
  • Not using SEO (or not passing a title prop)
  • Using <Head> from NextJS (however, our SEO component already adds a Head)

Why would you like this to be added removed:
This code is redundant (see also this) and should be removed or altered where necessary and substituted with SEO where applicable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chrispinkney chrispinkney

Labels
5 min fix Fixable in 5 minutes or less area: front-end area: nextjs Nextjs related issues type: enhancement New feature or request
Projects
None yet
Milestone
1.7 Release
2 participants
@humphd @chrispinkney