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

refactor: internal refactor for prepare new arch #3980

Merged

Conversation

YangJonghun
Copy link
Collaborator

Summary

refactor: internal refactor for prepare new arch

Motivation

Changes

Test plan

@freeboub
Copy link
Collaborator

freeboub commented Jul 8, 2024

need to change target brach !

@YangJonghun
Copy link
Collaborator Author

@freeboub
This PR is written so that it can be merged into master first.
I've just pre-named or pre-processed it to make easier to write this lib for new arch.

Copy link
Member

@KrzysztofMoch KrzysztofMoch left a comment

Choose a reason for hiding this comment

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

@freeboub I agree with @YangJonghun this PR mainly refactor names and types - it does not introduce any logic/behaviour change for user, so we can merge this into master. Also it will be easier to sync this later

@YangJonghun YangJonghun force-pushed the refactor/prepare-new-arch branch from 8ef9180 to 58d43d8 Compare July 10, 2024 15:53
@freeboub
Copy link
Collaborator

Ok for me, let's go !
Sorry for my mow availability these times..

@KrzysztofMoch KrzysztofMoch merged commit c2084c2 into TheWidlarzGroup:master Jul 12, 2024
11 checks passed
@YangJonghun YangJonghun deleted the refactor/prepare-new-arch branch July 12, 2024 08:32
@YangJonghun
Copy link
Collaborator Author

YangJonghun commented Jul 12, 2024

@freeboub @KrzysztofMoch
PR #3962 may cause problem with users who use 0.73.x (0.73.0's default Kotlin version is 1.9.0)

So, I add below commit within this PR.
3f96284

related issue : #3998

@KrzysztofMoch
Copy link
Member

Yeah I saw it - Thanks! I merged this and I will do 6.4.1 release

moskalakamil pushed a commit to moskalakamil/react-native-video that referenced this pull request Jul 12, 2024
* chore(js): fix typo

* refactor(js): refactor type code for codegen

* refactor(js): refactor Video component

- parse shutterColor value within JS
- remove internal fullscreen state

* chore(js): add deprecation warning comment

* fix(js): fix return type

* fix(js): fix import path

* refactor(android): apply changed API for new arch

* refactor(ios): apply changed API for new arch

* fix(ios): fix wrong name

* refactor: refactor VideoDecoderProperties

- rename and add wrapper

* refactor(android): Code fixes for backward compatibility with Kotlin
moskalakamil pushed a commit to moskalakamil/react-native-video that referenced this pull request Jul 17, 2024
* chore(js): fix typo

* refactor(js): refactor type code for codegen

* refactor(js): refactor Video component

- parse shutterColor value within JS
- remove internal fullscreen state

* chore(js): add deprecation warning comment

* fix(js): fix return type

* fix(js): fix import path

* refactor(android): apply changed API for new arch

* refactor(ios): apply changed API for new arch

* fix(ios): fix wrong name

* refactor: refactor VideoDecoderProperties

- rename and add wrapper

* refactor(android): Code fixes for backward compatibility with Kotlin
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.

3 participants