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

feat/upgrade-react-native-version #49

Merged
merged 19 commits into from
Jan 22, 2025
Merged

Conversation

OnestarLee
Copy link
Contributor

@OnestarLee OnestarLee commented Jan 16, 2025

External Contributions

This project is not yet set up to accept pull requests from external contributors.

If you have a pull request that you believe should be accepted, please contact
the Developer Relations team [email protected] with details
and we'll evaluate if we can setup a CLA to allow for the contribution.

For Internal Contributors

[CLNP-6191](https://sendbird.atlassian.net/browse/CLNP-6191)

Description Of Changes

Update sample react-native version to 0.76.0
Fixed a typo in the updateRemoteVideoView function on iOS

Types Of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply_

  • Bugfix
  • New feature
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, appearance (ex) Prettier)
  • Build configuration
  • Improvement (refactor code)
  • Test

@OnestarLee
Copy link
Contributor Author

react-native 0.76 으로 변경하면서 package.json 에 node 18 이상 사용하도록 추가 되었는데 circleci 의 node 버전을 올려야겠죠? @bang9
"engines": {
"node": ">=18"
},

@OnestarLee OnestarLee requested a review from bang9 January 16, 2025 11:28
@bang9
Copy link
Collaborator

bang9 commented Jan 17, 2025

react-native 0.76 으로 변경하면서 package.json 에 node 18 이상 사용하도록 추가 되었는데 circleci 의 node 버전을 올려야겠죠? @bang9 "engines": { "node": ">=18" },

넵 맞추면 좋을 것 �같습니다

@OnestarLee
Copy link
Contributor Author

OnestarLee commented Jan 18, 2025

sdk 와 sample에서 사용하는 typscript 버전이 달라서 tsc --noEmit 실행시 에러가 발생합니다.
type체크를 할때 sample을 별도로 처리하도록 변경하였는데 확인 부탁드릴께요

sample 에 tsconfig.json 추가

circle 스크립트 수정

yarn 도 버전업 되면서 --frozen-lockfile 제거되어 다음과 같이 변경하였습니다.
yarn install --cwd sample --immutable

yarn 1.22.22 버전으로 롤백하였습니다

@OnestarLee
Copy link
Contributor Author

OnestarLee commented Jan 18, 2025

rn 0.74 부터 yarn3 정식지원이라 upgrade helper 에 있는 yarn 3.6.4버전으로 변경했는데
"@sendbird/calls-react-native": "link:../", 가 제대로 동작하지 않습니다.

link 관련 기능이 yarn1 에 묵시적으로 허용되던게 yarn2+ 부터 제약조건등으로 workspace구성하지 않으면 안될수도 있다고하는데
요거 기존과 동일하게 yarn 1.22.22 로 변경하면 제대로 동작은 합니다.

혹시 방법 있을까요? 이것저것 다해봤는데 yarn3 버전에서는 link 가 제대로 동작을 안하네요
안되면 그냥 yarn 1 버전 사용하는것도 좋을것 같습니다 @bang9

일단 1.22.22 버전으로 롤백 하였습니다

@bang9
Copy link
Collaborator

bang9 commented Jan 20, 2025

rn 0.74 부터 yarn3 정식지원이라 upgrade helper 에 있는 yarn 3.6.4버전으로 변경했는데
"@sendbird/calls-react-native": "link:../", 가 제대로 동작하지 않습니다.

link 관련 기능이 yarn1 에 묵시적으로 허용되던게 yarn2+ 부터 제약조건등으로 workspace구성하지 않으면 안될수도 있다고하는데
요거 기존과 동일하게 yarn 1.22.22 로 변경하면 제대로 동작은 합니다.

혹시 방법 있을까요? 이것저것 다해봤는데 yarn3 버전에서는 link 가 제대로 동작을 안하네요
안되면 그냥 yarn 1 버전 사용하는것도 좋을것 같습니다 @bang9

일단 1.22.22 버전으로 롤백 하였습니다

아니면, 저희가 https://callstack.github.io/react-native-builder-bob/create 요 라이브러리를 통해서 스캐폴딩 프로젝트를 생성한건데요
최신 버전에서는 어떤식으로 처리가 되고 있는지 살펴봐도 좋을 것 같습니다.

@OnestarLee
Copy link
Contributor Author

OnestarLee commented Jan 21, 2025

create-react-native-library
이용해서 몇가지 테스트 해봤는데 아무래도 yarn2+ 이상에서 link 가 걸리는 제약이 있는것 같습니다.

root package(상위 폴더에 존재 @sendbird/calls-react-native) 에서
childe package(하위 폴더에 존재 @sendbird/calls-react-native-sample)
link 프로토콜이 제대로 동작하는데
childe 에서 root 를 link 걸면 제대로 동작하지 않고있네요
root 에서 childe 사용 : O
childe 에서 root 사용: X

관련해서 정확한 내용을 찾지는 못했는데, 일단 1.22.22 버전으로 두고 마무리 하겠습니다

@OnestarLee
Copy link
Contributor Author

안드로이드빌드후 시작할때 link 걸린 @sendbird/calls-react-native 를 제대로 찾지 못해서
react-native.config.js 추가하였습니다.
react native 버전업되면서 Autolinking 기능이 link 프로토콜로 처리된 모듈을 제대로 못찾는것 같습니다.
@bang9 리뷰 나온 내용 수정하고, 클린한 상태에서 iOS, Android 동작 되는것 모두 확인하였습니다.
요거 일단 머지만하고, 배포하는건 추후에 Android ringing 사운드 관련 수정 작업 되면 그거 포함해서 배포할까하는데 의견 부탁드립니다~

@bang9
Copy link
Collaborator

bang9 commented Jan 22, 2025

안드로이드빌드후 시작할때 link 걸린 @sendbird/calls-react-native 를 제대로 찾지 못해서
react-native.config.js 추가하였습니다.
react native 버전업되면서 Autolinking 기능이 link 프로토콜로 처리된 모듈을 제대로 못찾는것 같습니다.
@bang9 리뷰 나온 내용 수정하고, 클린한 상태에서 iOS, Android 동작 되는것 모두 확인하였습니다.
요거 일단 머지만하고, 배포하는건 추후에 Android ringing 사운드 관련 수정 작업 되면 그거 포함해서 배포할까하는데 의견 부탁드립니다~

넵 요거는 샘플 업데이트라서, 따로 배포는 필요하지 않을 것 같습니다!

Copy link
Collaborator

@bang9 bang9 left a comment

Choose a reason for hiding this comment

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

lgtm!

@OnestarLee OnestarLee merged commit 01efbed into main Jan 22, 2025
8 checks passed
@OnestarLee OnestarLee deleted the feat/upgrade-react-native-version branch January 22, 2025 00:46
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