Comments (8)
@RyanClementsHax I found it because I follow Micheal Shilman on Twitter. He was the one who convinced me to build the next router addon in the first place.
In short, yes I'd be happy to accept a PR and will review the one you sent shortly.
Im currently in the middle of launching my UI kit for a customer. https://lifetimeui.com but im happy to take a thorough look at the code once I wrap that up.
As for the PR I'm happy to accept upstream changes. I'm wondering how you solved the dependency problem with Contexts, from the 2 addons I maintain I've seen storybook clobber dependencies so when you try to bundle them into the addon they don't work which was why I went with the provider injection pattern. My original plan was to do what you're saying but I ran into issues with peer dependencies.
Thanks for the interest in collaboration.
from storybook-addon-next.
Yeah I'd be happy to collaborate! 😃
I originally built this to repackage your addon actually, but was having problems. I can't quite remember what problems I ran into but I think it was unrelated to your addon itself. Since then I've learned a lot and am willing to revisit getting it working.
For now I just published something just to get something shipped (better done than perfect), so thanks for the feedback.
As of right now specifically, I haven't packaged your addon because I'm working on building out a feature your addon doesn't support. I'm working on building in support for older versions of nextjs using webpack magic without having the user to provide any boilerplate or interact with nextjs internals (I'm trying to go for a "works out of the box" approach with this addon). I intend to support all the versions your addon supports. I was hoping to at least get support for those versions out the door soon just to make this addon usable for most of the nextjs ecosystem (should be done soon as I have a working POC and am now just cleaning up the code). After, however, would you like to have a feature like that merged upstream to your addon? I can make the changes in the form of a pull request so you can see the implementation diff if you would like.
By the way mind if I ask how you found out about this addon? I'm still amazed at how word spreads in open source especially since I only published this last week lol.
Also do u have any other feedback for this addon?
from storybook-addon-next.
Here is the pull request detailing what I had in mind:
Forgive its size (it includes the example repos that go along with it).
Among other refactors, this details the webpack magic approach I mentioned to get it working so that the developer doesn't have to reach into nextjs internals themselves to provide the addon the RouterContext
from storybook-addon-next.
@lifeiscontent I see you left comments on the pull request. I replied to them and am looking forward to your feedback. Is the general approach I took something you are comfortable including in your addon? I definitely agree it could use some massaging.
Also, I was thinking that the way forward could be the following:
- We massage the implementation provided by this addon in this repo
- I deploy those changes in this addon
- If nothing breaks, we make a pull request pulling in those changes to your addon
- You deploy those changes
- If nothing breaks, we update this addon to package your addon
- I deploy that change
What do you think?
from storybook-addon-next.
@RyanClementsHax Yeah, I think that approach could work. Sorry I haven't followed up on comments feel free to move forward, as we get closer to opening a PR on my addon I'll do a deeper audit then. Unfortunately I don't have a ton of time on my hands right now, do you have discord or something you communicate on frequently? Happy to start chatting and figuring out how to collaborate on this.
from storybook-addon-next.
I emailed you my discord user
from storybook-addon-next.
This issue has been automatically marked stale because it it received no activity for 60 days. If you wish to keep this open, please leave a comment. Thanks.
from storybook-addon-next.
This issue has been automatically closed because it received no activity for 60 days. If you think this was closed by accident, please leave a comment. Thanks.
from storybook-addon-next.
Related Issues (20)
- Does not work with `@nrwl/react/plugins/storybook` addon HOT 16
- Failed to load preset: /dist/preset.js HOT 3
- Next 12.2 newNextLinkBehavior support HOT 7
- I'm trying to use the addon with storybook and react, but can't build storybook once I installed the addon HOT 7
- Autoprefixer not working HOT 5
- Does not work with next/future/image HOT 3
- already written HOT 1
- Doesn't support AVIF format HOT 3
- Incopatibility with @storybook/builder-vite HOT 1
- Nextjs Image Error message Failed to parse src "static/media/public/logo.jpeg" on `next/image` HOT 3
- next 13 support HOT 11
- Error: NextRouter was not mounted. https://nextjs.org/docs/messages/next-router-not-mounted HOT 4
- Enabling react strict mode with this addon causes "Rendered more hooks than during the previous render." error. HOT 1
- Examples broken on next13, node 16.17.0 HOT 2
- Allow use of ES Module for NextJS config HOT 1
- Support Storybook v7 for upgrading HOT 9
- Nextjs Image Error message Failed to parse src "static/media/public/logo.jpeg" on next/legacy/image on Nextv13 HOT 1
- Image doesn't support the `loader` prop HOT 2
- Object.defineProperty(NextImage, '__esModule', descriptor) is no more needed from [email protected] HOT 4
- Next12.2 future/image support HOT 14
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from storybook-addon-next.