Skip to content

fix(ios): fix wrong react method definition (dismissFullscreenPlayer, presentFullscreenPlayer)#3338

Merged
KrzysztofMoch merged 1 commit intoTheWidlarzGroup:masterfrom
YangJonghun:fix/wrong-ios-header
Nov 8, 2023
Merged

fix(ios): fix wrong react method definition (dismissFullscreenPlayer, presentFullscreenPlayer)#3338
KrzysztofMoch merged 1 commit intoTheWidlarzGroup:masterfrom
YangJonghun:fix/wrong-ios-header

Conversation

@YangJonghun
Copy link
Copy Markdown
Contributor

@YangJonghun YangJonghun commented Nov 8, 2023

Update the changelog

fix(ios): fix wrong react method definition (dismissFullscreenPlayer, presentFullscreenPlayer)

Describe the changes

Corrected incorrect header file syntax to assign the correct key to VideoManager

@KrzysztofMoch KrzysztofMoch merged commit 7f49b56 into TheWidlarzGroup:master Nov 8, 2023
@YangJonghun YangJonghun deleted the fix/wrong-ios-header branch November 8, 2023 20:07
RCT_EXTERN_METHOD(dismissFullscreenPlayer:(nonnull NSNumber *)reactTag)

RCT_EXTERN_METHOD(dismissFullscreenPlayer
reactTag:(nonnull NSNumber *)reactTag)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should both of these be here?

Copy link
Copy Markdown
Contributor Author

@YangJonghun YangJonghun Jan 14, 2024

Choose a reason for hiding this comment

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

@jordangrant-stadiumlive
It looks my mistake.
could you create PR to correct this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

posted: #3474

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.

4 participants