-
Notifications
You must be signed in to change notification settings - Fork 54
Update stories to use new inputProvider #498
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove the defaultText
key from all stories except packages/components/psammead-sitewide-links/src/index.stories.jsx
please. Using the english translation is nicer than 'headline'
or 'caption'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pr has conflict which will need resolving
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to do Paragraph too, https://github.com/bbc/psammead/blob/latest/packages/components/psammead-paragraph/src/index.stories.jsx#L13
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
LGTM. Happy for this to be merged. |
Resolves #497
Resolves #495
Overall change: Update stories that use
inputProvider
to use version2.0.0
Code changes:
Caption
,SubHeading
,SitewideLinks
,InlineLink
,VisuallyHiddenText
andStoryPromo
stories to use@bbc/psammead-storybook-helpers@2.0.0