Skip to content

feat(PLAY-324): upgrade @flowplayer/player to latest canary, fix opts prop types#46

Merged
maximzavadskiy merged 14 commits intomainfrom
feat/PLAY-324/upgrade
May 10, 2024
Merged

feat(PLAY-324): upgrade @flowplayer/player to latest canary, fix opts prop types#46
maximzavadskiy merged 14 commits intomainfrom
feat/PLAY-324/upgrade

Conversation

@maximzavadskiy
Copy link
Contributor

@maximzavadskiy maximzavadskiy commented Apr 25, 2024

  • upgrade @flowplayer/player to 3.10.4-rc.4
  • fix opts prop types
  • add typespec tests

@maximzavadskiy maximzavadskiy force-pushed the feat/PLAY-324/upgrade branch from 8d52dc6 to f033636 Compare May 6, 2024 11:57
@maximzavadskiy maximzavadskiy changed the title Upgrade @flowplayer/player to 3.10.4-rc.1 feat(PLAY-324): upgrade @flowplayer/player to 3.10.4-rc.1, fix opts prop types May 6, 2024
@maximzavadskiy maximzavadskiy changed the title feat(PLAY-324): upgrade @flowplayer/player to 3.10.4-rc.1, fix opts prop types feat(PLAY-324): upgrade @flowplayer/player to 3.10.4-rc.4, fix opts prop types May 6, 2024
@maximzavadskiy maximzavadskiy changed the title feat(PLAY-324): upgrade @flowplayer/player to 3.10.4-rc.4, fix opts prop types feat(PLAY-324): upgrade @flowplayer/player to latest canary, fix opts prop types May 6, 2024
@maximzavadskiy maximzavadskiy requested a review from nnarhinen May 6, 2024 12:42
title: "Example title",
description: "Example description",
poster: { },
poster: "https://picsum.photos/200/300",
Copy link

@br3akzero br3akzero May 7, 2024

Choose a reason for hiding this comment

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

👍

Copy link

@br3akzero br3akzero left a comment

Choose a reason for hiding this comment

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

Great that you added a test for this!

@maximzavadskiy maximzavadskiy requested a review from br3akzero May 8, 2024 07:57
@maximzavadskiy
Copy link
Contributor Author

@rashadatjou good to merge?

@br3akzero
Copy link

@maximzavadskiy let me test this out locally and if everything is functioning I will give you the green light

Copy link

@br3akzero br3akzero left a comment

Choose a reason for hiding this comment

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

@maximzavadskiy the changes look great I just cleaned up the test/index.test-d.tsx file. This PR is good to be merged.

@maximzavadskiy maximzavadskiy merged commit 2bd5e21 into main May 10, 2024
@rj-wowza rj-wowza deleted the feat/PLAY-324/upgrade branch February 12, 2025 17:53
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.

2 participants

Comments