Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Update stories to use new inputProvider #498

Merged
21 commits merged into from
May 9, 2019
Merged

Update stories to use new inputProvider #498

21 commits merged into from
May 9, 2019

Conversation

ghost
Copy link

@ghost ghost commented May 3, 2019

Resolves #497
Resolves #495

Overall change: Update stories that use inputProvider to use version 2.0.0

Code changes:

  • Updated Caption, SubHeading, SitewideLinks, InlineLink, VisuallyHiddenText and StoryPromo stories to use @bbc/psammead-storybook-helpers@2.0.0

  • I have assigned myself to this PR and the corresponding issues
  • Tests added for new features
  • Test engineer approval

@ghost ghost added the ws-home Tasks for the WS Home Team label May 3, 2019
@ghost ghost self-assigned this May 3, 2019
@ghost ghost marked this pull request as ready for review May 3, 2019 15:14
Copy link
Contributor

@dr3 dr3 left a 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'

Copy link
Contributor

@dr3 dr3 left a 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

DenisHdz
DenisHdz previously approved these changes May 8, 2019
Copy link
Contributor

@DenisHdz DenisHdz left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@dr3 dr3 left a comment

Choose a reason for hiding this comment

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

andrew-nowak
andrew-nowak previously approved these changes May 8, 2019
Copy link
Contributor

@andrew-nowak andrew-nowak left a comment

Choose a reason for hiding this comment

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

lgtm 👍

DenisHdz
DenisHdz previously approved these changes May 8, 2019
Copy link
Contributor

@DenisHdz DenisHdz left a comment

Choose a reason for hiding this comment

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

Great work 👍

dr3
dr3 previously approved these changes May 8, 2019
@ghost ghost changed the title Update stories that use inputProvider Update stories to use new inputProvider May 9, 2019
@ghost ghost dismissed stale reviews from dr3, DenisHdz, and andrew-nowak via ef28556 May 9, 2019 08:35
Copy link
Contributor

@j-pendlebury j-pendlebury left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@jamesbrumpton
Copy link
Contributor

LGTM. Happy for this to be merged.

@ghost ghost merged commit abe2cc4 into latest May 9, 2019
@ghost ghost deleted the UpdateStories branch May 9, 2019 10:03
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-articles Tasks for the WS Articles Team ws-home Tasks for the WS Home Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update stories to use @bbc/psammead-storybook-helpers@2.0.0 InputProvider errors on components with children
6 participants