Open graph + favicons

[nerdlet]

Description

Fixes #170767031

Type of change

• New feature (non-breaking change which adds functionality)

Desktop Screenshots

Mobile Screenshots

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/codeforafrica/promisetracker/4dokvstmk
✅ Preview: https://promisetracker-git-feature-add-favicons.codeforafrica.now.sh

[nerdlet]

@kilemensi Not sure of this implementation per say, with the open graph stuff...let me know what you think

[kilemensi]

👍 @nerdlet ... I'm down with this approach.

[karimkawambwa]

Looking good

[nerdlet]

@kilemensi Ran into an issue when implementing DefaultSeo component. The issue is addressed here: [https://github.com/vercel/next.js/issues/10333] Fixed this by just using NextSeo instead of DefaultSeo

[kilemensi]

👍

[kilemensi]

@kilemensi Ran into an issue when implementing DefaultSeo component. The issue is addressed here: [https://github.com/zeit/next.js/issues/10333] Fixed this by just using NextSeo instead of DefaultSeo

I don't experience this @nerdlet ... This code with DefaultSeo works as advertised:

import { DefaultSeo } from 'next-seo';
import SEO from '../next-seo.config';

class PromiseTrackerApp extends App {
  componentDidMount() {
    // Remove the server-side injected CSS.
    const jssStyles = document.querySelector('#jss-server-side');
    if (jssStyles) {
      jssStyles.parentElement.removeChild(jssStyles);
    }
  }

  render() {
    const { Component, pageProps } = this.props;
    return (
      <>
        <MuiThemeProvider theme={theme}>
          <CssBaseline />
          <DefaultSeo {...SEO} />
          <Component {...pageProps} />
        </MuiThemeProvider>
      </>
    );
  }
}

[nerdlet]

@kilemensi It should be good now

[kilemensi]

👍 @nerdlet

[kilemensi]

🚀 🚀 @nerdlet

[kilemensi]

image: can be removed now.