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

psammead-storybook-helper - add timezone to withServicesKnob helper #3376

Merged
merged 2 commits into from
Apr 15, 2020

Conversation

sareh
Copy link
Contributor

@sareh sareh commented Apr 15, 2020

Part of #3358

This is needed to enable correct timezones in Storybook for the Radio Schedules component (PR #3377).

Overall change: Makes timezones available to Storybook knobs.
Note that the timezones added were obtained from the Simorgh Service Config files: e.g. for Afaan Oromoo

Code changes:

  • Added timezone
  • Updated tests

  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@sareh sareh added the ws-home Tasks for the WS Home Team label Apr 15, 2020
@sareh sareh self-assigned this Apr 15, 2020
@sareh sareh force-pushed the timezone-storybook-helper branch from aeb7f3a to 287eaef Compare April 15, 2020 06:36
@@ -34,6 +34,7 @@ export default ({
script,
locale,
dir = 'ltr',
timezone = 'GMT',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Setting a default timezone of 'GMT' since this is used when there is no clearly defined timezone for the service.

@sareh sareh linked an issue Apr 15, 2020 that may be closed by this pull request
1 task
@EinsteinNjoroge
Copy link
Contributor

EinsteinNjoroge commented Apr 15, 2020

Looks like storybook/chromatic is broken
Screen Shot 2020-04-15 at 10 24 48 AM

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

@sareh sareh merged commit 2426ee5 into latest Apr 15, 2020
@sareh sareh deleted the timezone-storybook-helper branch April 15, 2020 08:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-home Tasks for the WS Home Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Storybook stories not showing correct Timezone/numerals
3 participants