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

WSTEAM1-1542: Add live media to header - PRODUCTION #12239

Draft
wants to merge 94 commits into
base: latest
Choose a base branch
from

Conversation

shayneahchoon
Copy link
Contributor

@shayneahchoon shayneahchoon commented Dec 15, 2024

Resolves JIRA WSTEAM1-1521
Resolves JIRA WSTEAM1-1542

Overall changes

VERY IMPORTANT: Production video content will not work on test pages.

Adds live media to the header of a live page:
Screenshot 2024-12-15 at 11 59 40

Adds live tv media to the header of a live page:
Screenshot 2024-12-15 at 12 03 46

Code changes

  • Add a new liveMedia config for the MediaLoader component.
  • Adds a MediaLoader component to the Header of a live page.

Testing

  1. Go here: http://localhost.bbc.com:7081/mundo/live/c7dkx155e626t
  2. Storybook (note: media doesn't play) https://wsteam1-1521-live-video-front-end--5d28eb3fe163f6002046d6fa.chromatic.com/?path=/story/pages-live-page--with-live-stream

Helpful Links

Add Links to useful resources related to this PR if applicable.

Coding Standards

Repository use guidelines

@shayneahchoon shayneahchoon changed the title WSTEAM1-1521: Add live media to header WSTEAM1-1542: Add live media to header - PRODUCTION Jan 7, 2025
const MemoizedMediaPlayer = memo(MediaLoader);

const LiveMediaStream = ({ mediaCollection }: Props) => {
const { translations } = useContext(ServiceContext);
Copy link
Contributor

Choose a reason for hiding this comment

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

Could be worth adding an isLite check here to return null if its Lite. Just so we don't show a button to users that they can't interact with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants