better error message when api not reacheable on localhost

[xdamman]

[znarf]

Interesting using next config to forward environment variable to the client side.

Last time we had this need, we used _document.js, maybe your approach is better.

[xdamman]

Tests fail on

 FAIL  src/pages/__tests__/search.test.js
  ● Test suite failed to run

    TypeError: Cannot read property 'publicRuntimeConfig' of undefined

      11 | import getConfig from 'next/config'
      12 | 
    > 13 | const { publicRuntimeConfig } = getConfig();
         |         ^
      14 | 
      15 | class ErrorPage extends React.Component {
      16 | 

      at Object.<anonymous> (src/components/ErrorPage.js:13:9)
      at Object.<anonymous> (src/pages/search.js:19:1)
      at Object.<anonymous> (src/pages/__tests__/search.test.js:8:1)

Any idea how to fix that? /cc @HipsterBrown

[xdamman]

@znarf yeah, this comes from their documentation. I believed at the time there wasn't any official way to do it (so it was a hack) https://github.com/zeit/next.js#exposing-configuration-to-the-server--client-side

[znarf]

@xdamman have a look at that thread vercel/next.js#4024 (comment)

[znarf]

Let's make sure we're not logging errors to the console while testing. We want a clean console to easily see when something is wrong.

[znarf]

PR is too old and with too many conflicts.

Feel free to revisit these ideas in a new PR if that makes sense.