Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

[BPK-3800] Upgrade most Storybook dependencies #478

Merged
merged 1 commit into from
Apr 8, 2020

Conversation

shaundon
Copy link
Contributor

@shaundon shaundon commented Apr 8, 2020

This issue is preventing us from upgrading react-native-server but I was able to upgrade the rest. I'm now watching that issue so can try upgrading once it's resolved.

@backpackbot
Copy link

backpackbot commented Apr 8, 2020

Warnings
⚠️

package-lock.json was updated. Ensure that this was intentional.

Generated by 🚫 dangerJS against 9d0c01d

@shaundon shaundon force-pushed the BPK-3800-storybook-upgrade branch from 77a58e3 to 9d0c01d Compare April 8, 2020 16:38
@shaundon shaundon marked this pull request as ready for review April 8, 2020 16:38
// see: https://github.com/facebook/react-native/issues/18868#issuecomment-382671739
YellowBox.ignoreWarnings(['Warning: isMounted(...) is deprecated']);
};

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked and this doesn't seem to be an issue anymore, so I removed it.

const StorybookUI = getStorybookUI({ onDeviceUI: false });
const StorybookUI = getStorybookUI({
onDeviceUI: false,
asyncStorage: null,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#react-native-async-storage.

I tried to get it working but it caused a world of hurt, and for such a small feature I thought it's not worth the hassle. We can revisit this in the future if necessary.

@shaundon shaundon merged commit 4bbf5d9 into master Apr 8, 2020
@shaundon shaundon deleted the BPK-3800-storybook-upgrade branch April 8, 2020 17:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants