Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix issue https://github.com/storybookjs/react-native/issues/24 #10468

Merged
merged 1 commit into from
Apr 20, 2020

Conversation

fabien88
Copy link
Contributor

Issue: this commit 693f17e#diff-053b4430c28e7628f95731ed99a7723e from @core introduced a bug in react-native-server since v5.3.0-rc.6.
We can't see addon tabs anymore on the webpage as stated in this issue storybookjs/react-native#24

What I did

As @Gongreg stated, addon.js file is loaded only after provider.handleAPI(). It seems that is because entries order has been changed. This PR seems to put entries back in right order and makes addon tab reapear.

How to test

  • Is this testable with Jest or Chromatic screenshots?
    I don't know
  • Does this need a new example in the kitchen sink apps?
    No
  • Does this need an update to the documentation?
    No

@ndelangen ndelangen merged commit 7b7903a into storybookjs:next Apr 20, 2020
@ndelangen ndelangen added react-native bug patch:yes Bugfix & documentation PR that need to be picked to main branch labels Apr 20, 2020
@shilman shilman removed the patch:yes Bugfix & documentation PR that need to be picked to main branch label Jul 7, 2020
@dannyhw
Copy link
Member

dannyhw commented Aug 24, 2020

This fix would be really good to have in a 5.3.x release because it fixes a bug in react-native-server, any chance that would be possible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch react-native
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants