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

Creating addons section #91

Merged
merged 11 commits into from
Oct 5, 2019
Merged

Creating addons section #91

merged 11 commits into from
Oct 5, 2019

Conversation

ndelangen
Copy link
Member

This add a section to the guide on how to write addons

@ndelangen ndelangen added the enhancement New feature or request label May 16, 2019
@ndelangen ndelangen requested a review from tmeasday May 16, 2019 15:46
@ndelangen ndelangen self-assigned this May 16, 2019
Copy link
Member

@tmeasday tmeasday left a comment

Choose a reason for hiding this comment

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

So cool! I reckon we probably want to include some images of what things look like along the way.

@domyen probably has more to say

content/react/en/creating-addons.md Outdated Show resolved Hide resolved
@domyen
Copy link
Member

domyen commented Sep 7, 2019

@ndelangen @zol is this still active?

@zol
Copy link
Member

zol commented Sep 8, 2019

@domyen to the best of my knowledge, this was/is blocked on the 5.2 release, @ndelangen is that correct?

@ndelangen
Copy link
Member Author

correct, we need 5.2 to be released, which has the new hooks apis in it.

Copy link
Member

@zol zol left a comment

Choose a reason for hiding this comment

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

I'm good to merge this after my outstanding comments are addressed!

content/react/en/creating-addons.md Outdated Show resolved Hide resolved
content/react/en/creating-addons.md Outdated Show resolved Hide resolved
content/react/en/creating-addons.md Outdated Show resolved Hide resolved
content/react/en/creating-addons.md Outdated Show resolved Hide resolved
@ndelangen
Copy link
Member Author

@zol WDYT, mergeable?

@zol
Copy link
Member

zol commented Sep 30, 2019

@ndelangen It's almost good to go from my end, just need a better screenshot as per my remaining outstanding comment

@ndelangen
Copy link
Member Author

Screenshot 2019-10-04 at 14 15 18

@zol WDYT?

@ndelangen
Copy link
Member Author

Bit more stylised:
Screenshot 2019-10-04 at 14 16 58

# Conflicts:
#	content/intro-to-storybook/react/en/addons.md
#	content/intro-to-storybook/react/pt/addons.md
#	content/react/en/addons.md
#	content/react/en/using-addons.md
#	content/react/pt/addons.md
#	content/react/pt/using-addons.md
#	gatsby-config.js
@ndelangen
Copy link
Member Author

Screenshot 2019-10-04 at 14 56 02

@zol WDYT?

@zol
Copy link
Member

zol commented Oct 4, 2019

Looks great @ndelangen . I'm ready to have this merged and shipped!

@tmeasday
Copy link
Member

tmeasday commented Oct 5, 2019

@domyen merge when ready!

@ndelangen ndelangen merged commit 095f02c into master Oct 5, 2019
@ndelangen ndelangen deleted the creating-addons-section branch October 5, 2019 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants