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

Add lang to src of media player stories #2312

Merged
merged 4 commits into from
Oct 7, 2019

Conversation

pjlee11
Copy link
Contributor

@pjlee11 pjlee11 commented Oct 4, 2019

Part of bbc/simorgh#3342

Overall change: Updates the src values for the Media Player storybook examples`


  • 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

@pjlee11 pjlee11 added ws-articles Tasks for the WS Articles Team articles-av-epic ws-media The World Service media stream ws-media- LiveRadioV1 labels Oct 4, 2019
@pjlee11 pjlee11 self-assigned this Oct 4, 2019
Copy link
Contributor

@simonsinclair simonsinclair left a comment

Choose a reason for hiding this comment

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

All looks good in Storybook this end. 👍

I'm curious - what's the reason for removing test. from the audio assets?

@pjlee11
Copy link
Contributor Author

pjlee11 commented Oct 7, 2019

@simonsinclair not sure what you mean.

^ then you can add the lang (EG: /ko or /en-GB) and /amp as a suffix.

Edit: I've removed test. because ideally we want to use live data sources when they are available.

@pjlee11 pjlee11 merged commit ca18809 into latest Oct 7, 2019
@pjlee11 pjlee11 deleted the add-lang-to-media-player-stories-src branch October 7, 2019 15:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
articles-av-epic ws-articles Tasks for the WS Articles Team ws-media The World Service media stream ws-media- LiveRadioV1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants