FIX Update Storybook logo to the 5.1 OSS logo

[edwilde]

I use the storybook pattern lib fairly often, the new OSS logo has rolled out across the rest of dotorg, but appears to have been missed here.

I am not sure how permanent the silverstripe.org url is for the logo, should it be included in this repo instead?

[GuySartorelli]

Awesome, thank you for this. I'll let ci run and then merge.

[edwilde]

Quick note for any other product storybook noobs out there.

When trying to build initially I got this error.

info Found existing addon "@storybook/addon-controls", skipping.
info Found existing addon "@storybook/addon-actions", skipping.
info Found existing addon "@storybook/addon-controls", skipping.
info Found existing addon "@storybook/addon-actions", skipping.
info => Starting manager..
SB_CORE-COMMON_0004: Storybook could not index your stories.
Your main configuration somehow does not contain a 'stories' field, or it resolved to an empty array.

Please check your main configuration file and make sure it exports a 'stories' field that is not an empty array.

More info: https://storybook.js.org/docs/react/faq#can-i-have-a-storybook-with-no-local-stories

    at normalizeStories (./node_modules/@storybook/core-common/dist/index.js:21:2214)
    at iframe_webpack_config_default (./node_modules/@storybook/builder-webpack5/dist/presets/preview-preset.js:3:1890)
    at async starter (./node_modules/@storybook/builder-webpack5/dist/index.js:1:4658)

To fix this I had to adjust the projectRoot in ./.storybook/lib/storyTeller.js (line 111) and remove /../../../.

I then got this error towards the end of the build process

ModuleNotFoundError: Module not found: Error: Can't resolve 'constants'

This was fixed by adding an empty json object to the root called ./constants.json

echo "{}" > constants.json

Hope this helps someone in the future 🚀

[edwilde]

Thanks @GuySartorelli 👍