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

Typed channel api proposal #6189

Closed
wants to merge 1 commit into from
Closed

Typed channel api proposal #6189

wants to merge 1 commit into from

Conversation

benoitdion
Copy link
Member

Following up discussions in #5402, this a proposal for what a typed channel api could look like. Communication via channel is core to the storybook architecture. Adding type safety should help prevent bugs and help improve code discovery for new contributors.

Following up discussions in #5402, this a proposal for what a typed channel api could look like. Communication via channel is core to the storybook architecture. Adding type safety should help prevent bug and help improve code discovery for new contributors.
@github-actions
Copy link
Contributor

github-actions bot commented Mar 20, 2019

Fails
🚫 PR is marked with "do not merge" label.
🚫

PR is not labeled with one of: ["cleanup","doc-dependencies:update","BREAKING CHANGE","feature request","bug","documentation","maintenance","dependencies:update","dependencies","other"]

Generated by 🚫 dangerJS against b375528

@benoitdion benoitdion closed this Apr 14, 2019
@github-actions
Copy link
Contributor

Fails
🚫 PR is marked with "do not merge" label.
🚫

PR is not labeled with one of: ["cleanup","doc-dependencies:update","BREAKING CHANGE","feature request","bug","documentation","maintenance","dependencies:update","dependencies","other"]

Generated by 🚫 dangerJS against b375528

@shilman
Copy link
Member

shilman commented Apr 15, 2019

@benoitdion why did u close?

@benoitdion
Copy link
Member Author

this was more of a proposal/design doc than something ready to be merged. We can keep open for reference until one of us gets a chance to work on this if you think that's useful though.

@stof stof deleted the api-types branch May 25, 2022 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants