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

Feature - Storybook 8.0 #76

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

gentslava
Copy link

@gentslava gentslava commented Mar 15, 2024

@EldinZaimovic
Copy link

I hope this will be merged soon :)

@jmcpeak
Copy link

jmcpeak commented Apr 16, 2024

Any movement on merging this PR? Be nice to see this warning go away:

WARN - @storybook/[email protected]
WARN  Repo: https://github.com/storybookjs/storybook/tree/next/code/deprecated/addons
WARN 
WARN 
WARN Please consider updating your packages or contacting the maintainers for compatibility details.
WARN For more on Storybook 8 compatibility, see the linked GitHub issue:
WARN https://github.com/storybookjs/storybook/issues/26031

@aestradaes
Copy link

Very nice! I was just looking for this, any idea when it will get merged?

@gone-skiing
Copy link

Bump... Could we get this merged to unblock upgrading to Storybook 8, please and thank you!

@jsoulet
Copy link

jsoulet commented Jul 3, 2024

I'm also waiting for the PR to be merged 🙂
Thank you

@bhishp
Copy link
Collaborator

bhishp commented Jul 8, 2024

Firstly, apologies for my delay in addressing this PR, it seems a few of you are waiting on it to be merged. I am no longer developing as part of my day job, so if anyone is interested in adopting this project as a maintainer then please do shout.

With regards to the PR, have you tested the functionality @gentslava? I have tried building and running your branch but I'm not actually seeing it behave as expected (see screenshot, no values populated in the panel)

image

As mentioned, I'm not developing anymore and not familiar with the sb8 migration requirements

@eldinz-getin
Copy link

any update on this ?
Thx

@gentslava
Copy link
Author

Hey, @bhishp!
Thanks for waiting while I was on vacation.
I didn't notice this extension. I need some time to figure out how it works and improve it.
I'll be back!

@EldinZaimovic
Copy link

Any update about this one?

@bhishp
Copy link
Collaborator

bhishp commented Nov 14, 2024

Any volunteers to fork and fix the PR?

@Crownpack07
Copy link
Contributor

Crownpack07 commented Nov 15, 2024

I have created an updated PR which seems to be working, please have a look at it and let me know #77

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants