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

친구 탭 React Native 베이스 브랜치 #182

Merged
merged 45 commits into from
Sep 10, 2023
Merged

친구 탭 React Native 베이스 브랜치 #182

merged 45 commits into from
Sep 10, 2023

Conversation

JuTaK97
Copy link
Collaborator

@JuTaK97 JuTaK97 commented Jul 2, 2023

플로우

  1. 앱 실행
    • ReactNativeBundleManager에서 config의 fetch 완료 대기
    • RemoteConfig에서 token collect 시작하고, 값을 받으면 getRemoteConfig 후 channel에 전송
  2. ReactNativeBundleManager가 config fetch를 확인하고 token collect 시작
  3. token이 있으면, config에서 받은 src로 번들 파일 유효성 확인
    번들 파일이 없거나 손상됐으면 다운로드
  4. 번들 파일로 ReactRootView 세팅, 유저가 친구탭 이동하면 띄우기
  5. 로그아웃 후 다시 로그인 시 2번에서 새 token collect 후 3~4 반복

@JuTaK97 JuTaK97 requested a review from a team as a code owner July 2, 2023 16:05
@JuTaK97 JuTaK97 changed the title React Native 베이스 브랜치 친구 탭 React Native 베이스 브랜치 Aug 11, 2023
@JuTaK97 JuTaK97 force-pushed the jutak/rn branch 5 times, most recently from 61c54de to 6e0b629 Compare August 15, 2023 16:40
@JuTaK97 JuTaK97 force-pushed the jutak/rn branch 3 times, most recently from 999e99a to 3e558c6 Compare September 8, 2023 14:09
Comment on lines 85 to 89
val isDarkMode = if (userViewModel.themeMode.value == ThemeMode.AUTO) {
isSystemDarkMode(applicationContext)
} else {
userViewModel.themeMode.value == ThemeMode.DARK
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

다크모드 여부 구하는 로직

Copy link
Collaborator

Choose a reason for hiding this comment

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

화면 모드 바꾸고 친구탭 들어가면 반영이 안됨.... themMode를 collect 해서 ReactNativeBundleManager를 갈아끼워야 하나??
사진은 친구탭 -> 화면 모드 라이트에서 다크로 변경 -> 다시 친구탭 들어간 화면

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

그거 나도 좀 고민했는데 걍 대응안해도 되지않을까

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아니다 걍 대응해야겠다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

PendingIntent.FLAG_ONE_SHOT,
PendingIntent.FLAG_ONE_SHOT or PendingIntent.FLAG_IMMUTABLE,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

버전 올리고 컴파일 막혀서 수정
MyFirebaseMessegingServie 안쓰긴 하는데, 지우긴 좀 아까워서 냅둠

Comment on lines +51 to +72
override fun getReactNativeHost(): ReactNativeHost {
return object : ReactNativeHost(this) {
override fun getUseDeveloperSupport(): Boolean = BuildConfig.DEBUG

override fun getPackages(): List<ReactPackage> = listOf(
MainReactPackage(),
RNScreensPackage(),
RNGestureHandlerPackage(),
RNCPickerPackage(),
SafeAreaContextPackage(),
ReanimatedPackage(),
SvgPackage(),
)

override fun getJSMainModuleName(): String = "friends"

override fun getJavaScriptExecutorFactory(): JavaScriptExecutorFactory {
return HermesExecutorFactory()
}
}
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ReactApplication 상속 구현부

init {
CoroutineScope(Dispatchers.IO).launch {
remoteConfig.waitForFetchConfig()
token.filter { it.isNotEmpty() }.collectLatest {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이렇게 하면 로그아웃 했다가 로그인 했을 때 ReactView랑 ReactInstanceManager를 새로 설정하게 된다.

Comment on lines +38 to +42
if (BuildConfig.DEBUG && FlipperUtils.shouldEnableFlipper(this)) {
val client = AndroidFlipperClient.getInstance(this)
client.addPlugin(InspectorFlipperPlugin(this, DescriptorMapping.withDefaults()))
client.start()
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Flipper 설정

JuTaK97 and others added 23 commits September 10, 2023 21:23
commit e3f5ca2d1e08bf9afb56477290a86cac24b90e1d
Author: 양주현 <didwngus01@naver.com>
Date:   Thu Sep 7 21:20:09 2023 +0900

    prod 빌드 되게 하기

    test

    17

    고쳐야 하는 것

    aab 체크

    abi false

    버전 롤백

    뭐가 문제지

    live도

    17

    Revert "17"

    This reverts commit 1a5171f.

    Revert "live도"

    This reverts commit 14114f2.

    Revert "뭐가 문제지"

    This reverts commit 0b07a3b.

    Revert "버전 롤백"

    This reverts commit 223b137.

    Revert "abi false"

    This reverts commit 668523b.

    Revert "aab 체크"

    This reverts commit 7d7b025.

    revert 후 cd만 셋팅

    aab 받아보기

    경로 수정
@JuTaK97 JuTaK97 enabled auto-merge (squash) September 10, 2023 12:24
@JuTaK97 JuTaK97 disabled auto-merge September 10, 2023 12:24
@JuTaK97 JuTaK97 enabled auto-merge (squash) September 10, 2023 12:24
@JuTaK97 JuTaK97 merged commit b58cf18 into develop Sep 10, 2023
3 checks passed
@JuTaK97 JuTaK97 deleted the jutak/rn branch September 10, 2023 12:34
@JuTaK97 JuTaK97 mentioned this pull request Sep 18, 2023
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