search

ProvidenceGeeks / website-frontend

UI frontend repository for the Providence Geeks website
https://www.pvdgeeks.org
7 stars 9 forks source link

add open graph tags #164

Closed thescientist13 closed 6 years ago

thescientist13 commented 6 years ago

Related Issue

resolves #163

Summary of Changes

  1. Added relevant Open Graph tags for the application as default in index.html

TODO

  1. [ ] Implement dynamic tag generation for Blog Post Details page(s)
thescientist13 commented 6 years ago

So while Helmet is good for dynamically inserting tags, it won't work with something like Twitter cards and other crawlers where the tags must be there as part of the initial HTML download.

screen shot 2018-05-23 at 3 26 07 pm

Gatsby (from my example in the issue) generates all this at build time, unlike what I was trying to do here

For now though, we get this, which is nice I think.

screen shot 2018-05-23 at 3 31 02 pm screen shot 2018-05-23 at 3 33 55 pm

Note: this change is in stage if you feel like testing it out.