-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feature] 내 상점 페이지 이벤트/공지 수정 삭제 #314
Conversation
…nto feature/business/store_detail # Conflicts: # business/src/main/java/in/koreatech/business/feature/store/EventScreen.kt # data/src/main/java/in/koreatech/koin/data/api/auth/OwnerAuthApi.kt # data/src/main/java/in/koreatech/koin/data/source/remote/OwnerRemoteDataSource.kt
# Conflicts: # business/src/main/java/in/koreatech/business/feature/store/EventScreen.kt # business/src/main/java/in/koreatech/business/feature/store/MyStoreDetailState.kt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생했습니다. 👍
@@ -116,7 +116,18 @@ object BusinessAuthNetworkModule { | |||
): Interceptor { | |||
return Interceptor { chain: Interceptor.Chain -> | |||
runBlocking { | |||
val ownerAccessToken = tokenLocalDataSource.getOwnerAccessToken() ?: "" | |||
val ownerAccessToken = | |||
"eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzM4NCJ9.eyJpZCI6NTU0OCwiZXhwIjoxNzE5ODg2Nzc4fQ.5LjbuPxi1G0aQoxtvWjB3ajpd28k6kjjCp4I199fkt3C8cRa40zxcjJwS91X0pSi" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
access token 하드코딩으로 넣은 이유가 뭔가요? 테스트하다가 실수로 넣은건가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
절대로 토큰을 commit 하지 마세요. 시크릿 키 포함
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트하다가 실수로 커밋했습니다..😣
business/src/main/java/in/koreatech/business/feature/store/AppBar.kt
Outdated
Show resolved
Hide resolved
business/src/main/java/in/koreatech/business/feature/store/OwnerStoreDialog.kt
Outdated
Show resolved
Hide resolved
business/src/main/java/in/koreatech/business/feature/store/storedetail/event/EventItem.kt
Outdated
Show resolved
Hide resolved
domain/src/main/java/in/koreatech/koin/domain/usecase/business/DeleteOwnerEventsUseCase.kt
Outdated
Show resolved
Hide resolved
business/src/main/java/in/koreatech/business/feature/store/storedetail/event/EventScreen.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 하셨습니다! comment 확인해주세요 :)
@@ -116,7 +116,18 @@ object BusinessAuthNetworkModule { | |||
): Interceptor { | |||
return Interceptor { chain: Interceptor.Chain -> | |||
runBlocking { | |||
val ownerAccessToken = tokenLocalDataSource.getOwnerAccessToken() ?: "" | |||
val ownerAccessToken = | |||
"eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzM4NCJ9.eyJpZCI6NTU0OCwiZXhwIjoxNzE5ODg2Nzc4fQ.5LjbuPxi1G0aQoxtvWjB3ajpd28k6kjjCp4I199fkt3C8cRa40zxcjJwS91X0pSi" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
절대로 토큰을 commit 하지 마세요. 시크릿 키 포함
business/src/main/java/in/koreatech/business/feature/store/storedetail/MyStoreDetailScreen.kt
Outdated
Show resolved
Hide resolved
…erStoreDialog.kt Co-authored-by: Jokwanhee <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 수고하셨습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 👍
개요
작업 상세내용
결과 화면