Feat/week xml 07 #19

cacaocoffee · 2024-06-07T03:45:28Z

🛠 Related issue

[feat] 7주차 과제_xml #18

✏️ Work Description

레포지토리 패턴적용
usecase분리

📸 Screenshot

4.mp4

😅 Uncompleted Tasks

📢 To Reviewers

usecase 분리 이렇게하는게 맞는지 궁금합니다.
invoke로 별도 함수 선언 없이하는게 좋을지 궁금합니다.
7주차 동안 부족한 저에게 리뷰 많이 달아주셔서 다들 감사해요

✅ Check List

Code/Git 컨벤션은 지켰나요?
PR올리기 전에 검토했나요?
코드리뷰 반영했나요?

…-xml-07

librarywon

안녕하세요 명석님 마지막 주차 코드리뷰를 맡게된 명예 OB 서재원입니다!
저번에 리뷰 할 때에 비해 폭팔적으로 성장하신 모습을 본 것 같아서 정말 존경을 표합니다.
34기 YB분들은 정말 다들 실력이 출중하신 것 같군요, 안드로이드 앱 아키텍처와 세세한 코틀린 문법의 개념을 채우시면 더 괴물같은 개발자가 되실 것 같습니다.
세미나 완주하시느라 정말 고생하셨습니다. 행복 앱잼 하시길 👍

librarywon · 2024-06-09T10:54:24Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

+import androidx.lifecycle.viewModelScope
 import com.sopt.now.data.model.User
-import com.sopt.now.util.StringNetworkError.FAIL_ERROR
-import com.sopt.now.util.StringNetworkError.LOGIN
-import com.sopt.now.util.StringNetworkError.SERVER_ERROR
+import com.sopt.now.domain.entity.request.AuthRequestModel
+import com.sopt.now.domain.usecase.LogInUseCase


지난 리뷰에서도 말씀드렸던 문제인데
지금 안드로이드 앱 아키텍처를 적용중이시라면 ui -> domain -> data 형식으로 참조가 되어야할 것입니다.
하지만 data의 model를 불러와서 사용중이니 ui가 domain과 data를 모두 참조하는 모양새이죠
어떻게 이 문제를 해결할지 고민해보시는 것도 좋을 것 같습니다.

너무 잘못쓰고있었네요..
말씀해주신 주제와 아키텍처 여러 관련된 자료 찾아보고 공부한다고 답변을 너무 늦게 달았네요

제대로 한지는 모르지만 clean 아키텍처에 대해서 찾아 공부하고 적용을 하려 노력을 했습니다.

앱잼에 제대로 들어가전에 구글권장아키텍처도 제대로 확인해보고 가려구요 감사합니다.

멋진 성장 기대하겠습니다!

librarywon · 2024-06-09T10:55:50Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

    private val _loginState = MutableLiveData<UiState<User>>()
    val loginState = _loginState


사소하지만 변수의 타입을 명시적으로 선언해주는 것이 좋습니다.

넵 감사합니다! 타입 명시하는 것 잘 지켜보도록 하겠습니다! 사실 명시하는 게 좋은 건 줄 몰랐어요

컴퓨터관점에서 추론하는 것보다 명시되어있는걸 가져다 쓰는게 좋겠다는 생각입니다.

librarywon · 2024-06-09T10:56:42Z

app/src/main/java/com/sopt/now/ui/login/LoginActivity.kt

+    private fun getLoginRequest() =
+        AuthRequestModel(
            authenticationId = binding.etLoginId.text.toString(),
-            password = binding.etLoginPassword.text.toString()
+            password = binding.etLoginPassword.text.toString(),
+            nickname = "",
+            phone = ""
        )


데이터 바인딩을 사용하면 이 귀찮은 과정을 회피할 수도 있습니다!

librarywon · 2024-06-09T10:59:22Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModelFactory.kt

+class LoginViewModelFactory : ViewModelProvider.Factory {
+    override fun <T : ViewModel> create(modelClass: Class<T>): T {
+        if (modelClass.isAssignableFrom(LoginViewModel::class.java)) {
+            val repository =
+                AuthRepositoryImpl(ServicePool.authService)
+            val logInUseCase = LogInUseCase(repository)
+            return LoginViewModel(logInUseCase) as T
+        } else {
+            throw IllegalArgumentException("Failed to create ViewModel: ${modelClass.name}")
+        }
+    }
+}


ViewModelFactory를 사용하다보면 뷰모델마다 뷰모델펙토리가 생성되는 불편함이 생기는 것을 느끼셨을 겁니다.
프로젝트에 적용할지는 모르겠지만 Hilt로 마이그레이션하면서 자동 DI가 어떤 이점을 가져오는지 알아보시면 정말 도움이 많이 될 것 같습니다.

맞습니다.. 저도 그렇게 했더니 번거롭더라구요 저도 Hilt 다음에 꼭 적용해봐야겠습니다!!

우선 viewmodel 팩토리 하나로 작업을 했습니다. hilt에 대해선 공부하고 적용해보는 단계에 있어요.
피와살이 되는 조언 감사합니다..

librarywon · 2024-06-09T11:00:11Z

app/src/main/java/com/sopt/now/ui/main/MainViewModel.kt

+        viewModelScope.launch {
+            runCatching {
+                infoService.getUserInfo(userid)
+            }.onSuccess { response ->


여기는 repository 패턴이 적용되어 있지 않은건가요?

넵 당시에 login과 signup을 하면서 제가 공부한거에 대한 확신이 없었어요...
뭔가 이렇게 짜는 것이 맞는건가 고민이 들면서 확실히 하고 적용을 하기로 했었습니다!
지금은 추가해서 넣어두었어요!

librarywon · 2024-06-09T11:03:51Z

app/src/main/java/com/sopt/now/domain/entity/request/AuthRequestModel.kt

+data class AuthRequestModel(
+    val authenticationId: String,
+    val password: String,
+    val nickname: String,
+    val phone: String,
+) {
+    fun toRequestSignup(): RequestSignUpDto =
+        RequestSignUpDto(authenticationId, password, nickname, phone)
+
+    fun toRequestLogin(): RequestLoginDto = RequestLoginDto(authenticationId, password)
+    fun toUserWithUserId(userid: String): User =
+        User(
+            id = authenticationId,
+            password = password,
+            nickname = nickname,
+            phoneNumber = phone,
+            userId = userid
+        )
+}


AuthRequestModel를 쓰시게된 이유가 궁금합니다!

data 영역에서 ui 에 사용하기에 중간다리로 생각하고 추가했습니다.
저 당시에 의존관계가 헷갈려 너무 잘못짜고 넣었네요.. 재원님 리뷰 덕분에 왜 사용하는가에 대한 본질적인 고민을 학 수정했습니다 감사해요

참신한 접근인 것 같아요 이런 본인만의 접근도 좋다고 생각합니다

librarywon · 2024-06-09T11:05:11Z

app/src/main/java/com/sopt/now/domain/repository/AuthRepository.kt

+import retrofit2.Response
+
+interface AuthRepository {
+    suspend fun logIn(authData: AuthRequestModel): Result<Response<BaseResponse<Unit>>>


login은 통상적으로 한단어로 인식합니다!

감사합니다 뭔가.. 갑자기 헷갈렸어요..

librarywon · 2024-06-09T11:06:28Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+    companion object {
+        const val MESSAGE = "message"
+        const val LOCATION = "location"
+    }


private를 따로 붙이시지 않으셨는데 다른 곳에서 사용하는 것 일까요?

없습니다 private 붙이는걸 까먹었네요 감사합니다.

hyoeunjoo

코드리뷰 항상 많이 받기만 하고 양질의 코리를 못 달아드린 것 같아서 속상해요ㅠㅠ
(다음 기수에 제가 더 발전해서 계속 달아드리고싶네요 ㅎ)
이번 과제도 수고 많으셨습니다!!

hyoeunjoo · 2024-06-09T13:15:52Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModelFactory.kt

+class LoginViewModelFactory : ViewModelProvider.Factory {
+    override fun <T : ViewModel> create(modelClass: Class<T>): T {
+        if (modelClass.isAssignableFrom(LoginViewModel::class.java)) {
+            val repository =
+                AuthRepositoryImpl(ServicePool.authService)
+            val logInUseCase = LogInUseCase(repository)
+            return LoginViewModel(logInUseCase) as T
+        } else {
+            throw IllegalArgumentException("Failed to create ViewModel: ${modelClass.name}")
+        }
+    }
+}


맞습니다.. 저도 그렇게 했더니 번거롭더라구요 저도 Hilt 다음에 꼭 적용해봐야겠습니다!!

hyoeunjoo · 2024-06-09T13:20:08Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+
+class LogInUseCase(private val authRepository: AuthRepository) {
+    suspend fun execute(request: AuthRequestModel): UiState<User> {
+        return withContext(Dispatchers.IO) {
+            try {
+                val result = authRepository.logIn(request)
+                result.fold(
+                    onSuccess = { response ->
+                        if (response.isSuccessful) {
+                            UiState.Success(
+                                User(
+                                    request.authenticationId,
+                                    request.password,
+                                    "",
+                                    "",
+                                    userId = response.headers()[LOCATION].toString()
+                                )
+                            )


아.. UseCase 분리.. 잊고 있었는데!!!
멋져요.. 가독성이 훨 좋네요

아니예요 ㅠㅜ 분리는했는데 너무 잘못해서.. 바뀐 코드처럼 해야하는걸로 알고있어요

hyoeunjoo · 2024-06-09T13:20:55Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

+        viewModelScope.launch {
+            _loginState.value = logInUseCase.execute(request)
+        }


속이 막 시원해지네요

이러면 안된대요 ㅠ

OliviaYJH

수고하셨습니다!

OliviaYJH · 2024-06-09T14:39:58Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+
+class LogInUseCase(private val authRepository: AuthRepository) {
+    suspend fun execute(request: AuthRequestModel): UiState<User> {
+        return withContext(Dispatchers.IO) {


withContext로도 코루틴을 실행할 수 있군요! 알아가요!!

넵 그런데 usecase에서 이렇게 쓰면안돼요! domain 영역에 서버통신이랑 여러 과정이 들어가게 되어서 잘 못 짠 코드입니다!

chanubc

한 학기 동안 고생 많으셨습니다!!
너무나 열심히 잘해주어서 고마워요ㅠㅠㅠㅠ

자세한 사항은 따로 개인톡 드릴게요!

chanubc · 2024-06-09T18:06:10Z

app/src/main/java/com/sopt/now/domain/usecase/SignUpUseCase.kt

일단 이 로직은 data영역에 있어야 하는 로직입니다!
repository패턴에 의하면 data영역에서 서버 통신과 같은 로직을 담당 하게 되는데
domain의 이러한 서버통신 로직을 담당하는 것이 아닌 repository 인터페이스를 통해 데이터를 요청하는 역할만 수행하게 됩니다. 실제 구현은 data에서 하게되고 이를 의존성 역전이라고 부릅니다.

domain의 usecase는 이러한 interface를 참조하여 특정 비즈니스 로직을 캡슐화 합니다.

지금 data 레이어의 분리가 목적인지 혹은 클린아키텍쳐식의 구조가 목적인지 확실하게 정하고 다시 리팩토링 해보면 좋을것 같아요!!

정말 감사합니다
클린아키텍처를 적용해보자! 라고 하긴했는데 막상 하면서 이상한거 같더라구요...
제가 하는 방면이 맞나 싶기도 했었는데 덕분에 의문점을 많이 해결하고 작업할 수 있었어요.
지금 수정한 방안이 정답이 아닐 수는 있더라도 덕분에 많은 걸 알게 되었어요 감사합니다.
이후에 자세한 설명 덕분에 현재 공부하고 적용을 해봤어요 datasource 로 분리하고 hilt 적용도 노력해보겠습니다..

chanubc · 2024-06-09T18:08:28Z

app/src/main/java/com/sopt/now/domain/entity/request/AuthRequestModel.kt

+data class AuthRequestModel(
+    val authenticationId: String,
+    val password: String,
+    val nickname: String,
+    val phone: String,
+) {
+    fun toRequestSignup(): RequestSignUpDto =
+        RequestSignUpDto(authenticationId, password, nickname, phone)
+
+    fun toRequestLogin(): RequestLoginDto = RequestLoginDto(authenticationId, password)
+    fun toUserWithUserId(userid: String): User =
+        User(
+            id = authenticationId,
+            password = password,
+            nickname = nickname,
+            phoneNumber = phone,
+            userId = userid
+        )
+}


domain에서 dto를 참조하는 것이 아닌 data에서 dto를 참조해야 합니다!

넵 감사합니다. 안그래도 코드 짜면서 뭔가 잘못되었다는 생각이 들어서 고민많이 했었는데 덕분에 길을 찾을 수 있었어요

cacaocoffee

너무 나도 늦은 답변 드려 죄송합니다. 다들 부족한 저에게도 comment와 도움 주셔서 고마워요 솝트 활동하는 동안 여러분의 코드리뷰가 너무나도 큰 도움이 되었습니다.

다시 한번 너무 감사드립니다. 큰 도움 받았어요.

cacaocoffee · 2024-06-11T17:45:20Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+    companion object {
+        const val MESSAGE = "message"
+        const val LOCATION = "location"
+    }


없습니다 private 붙이는걸 까먹었네요 감사합니다.

cacaocoffee · 2024-06-14T21:51:39Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+
+class LogInUseCase(private val authRepository: AuthRepository) {
+    suspend fun execute(request: AuthRequestModel): UiState<User> {
+        return withContext(Dispatchers.IO) {


넵 그런데 usecase에서 이렇게 쓰면안돼요! domain 영역에 서버통신이랑 여러 과정이 들어가게 되어서 잘 못 짠 코드입니다!

cacaocoffee · 2024-06-14T21:51:55Z

app/src/main/java/com/sopt/now/domain/usecase/LogInUseCase.kt

+
+class LogInUseCase(private val authRepository: AuthRepository) {
+    suspend fun execute(request: AuthRequestModel): UiState<User> {
+        return withContext(Dispatchers.IO) {
+            try {
+                val result = authRepository.logIn(request)
+                result.fold(
+                    onSuccess = { response ->
+                        if (response.isSuccessful) {
+                            UiState.Success(
+                                User(
+                                    request.authenticationId,
+                                    request.password,
+                                    "",
+                                    "",
+                                    userId = response.headers()[LOCATION].toString()
+                                )
+                            )


아니예요 ㅠㅜ 분리는했는데 너무 잘못해서.. 바뀐 코드처럼 해야하는걸로 알고있어요

cacaocoffee · 2024-06-14T21:53:41Z

app/src/main/java/com/sopt/now/domain/entity/request/AuthRequestModel.kt

+data class AuthRequestModel(
+    val authenticationId: String,
+    val password: String,
+    val nickname: String,
+    val phone: String,
+) {
+    fun toRequestSignup(): RequestSignUpDto =
+        RequestSignUpDto(authenticationId, password, nickname, phone)
+
+    fun toRequestLogin(): RequestLoginDto = RequestLoginDto(authenticationId, password)
+    fun toUserWithUserId(userid: String): User =
+        User(
+            id = authenticationId,
+            password = password,
+            nickname = nickname,
+            phoneNumber = phone,
+            userId = userid
+        )
+}


data 영역에서 ui 에 사용하기에 중간다리로 생각하고 추가했습니다.
저 당시에 의존관계가 헷갈려 너무 잘못짜고 넣었네요.. 재원님 리뷰 덕분에 왜 사용하는가에 대한 본질적인 고민을 학 수정했습니다 감사해요

cacaocoffee · 2024-06-14T21:54:32Z

app/src/main/java/com/sopt/now/domain/entity/request/AuthRequestModel.kt

+data class AuthRequestModel(
+    val authenticationId: String,
+    val password: String,
+    val nickname: String,
+    val phone: String,
+) {
+    fun toRequestSignup(): RequestSignUpDto =
+        RequestSignUpDto(authenticationId, password, nickname, phone)
+
+    fun toRequestLogin(): RequestLoginDto = RequestLoginDto(authenticationId, password)
+    fun toUserWithUserId(userid: String): User =
+        User(
+            id = authenticationId,
+            password = password,
+            nickname = nickname,
+            phoneNumber = phone,
+            userId = userid
+        )
+}


넵 감사합니다. 안그래도 코드 짜면서 뭔가 잘못되었다는 생각이 들어서 고민많이 했었는데 덕분에 길을 찾을 수 있었어요

cacaocoffee · 2024-06-14T22:11:31Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

    private val _loginState = MutableLiveData<UiState<User>>()
    val loginState = _loginState


넵 감사합니다! 타입 명시하는 것 잘 지켜보도록 하겠습니다! 사실 명시하는 게 좋은 건 줄 몰랐어요

cacaocoffee · 2024-06-14T22:11:41Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

+        viewModelScope.launch {
+            _loginState.value = logInUseCase.execute(request)
+        }


이러면 안된대요 ㅠ

cacaocoffee · 2024-06-14T22:12:25Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModelFactory.kt

+class LoginViewModelFactory : ViewModelProvider.Factory {
+    override fun <T : ViewModel> create(modelClass: Class<T>): T {
+        if (modelClass.isAssignableFrom(LoginViewModel::class.java)) {
+            val repository =
+                AuthRepositoryImpl(ServicePool.authService)
+            val logInUseCase = LogInUseCase(repository)
+            return LoginViewModel(logInUseCase) as T
+        } else {
+            throw IllegalArgumentException("Failed to create ViewModel: ${modelClass.name}")
+        }
+    }
+}


우선 viewmodel 팩토리 하나로 작업을 했습니다. hilt에 대해선 공부하고 적용해보는 단계에 있어요.
피와살이 되는 조언 감사합니다..

cacaocoffee · 2024-06-14T22:13:15Z

app/src/main/java/com/sopt/now/ui/main/MainViewModel.kt

+        viewModelScope.launch {
+            runCatching {
+                infoService.getUserInfo(userid)
+            }.onSuccess { response ->


넵 당시에 login과 signup을 하면서 제가 공부한거에 대한 확신이 없었어요...
뭔가 이렇게 짜는 것이 맞는건가 고민이 들면서 확실히 하고 적용을 하기로 했었습니다!
지금은 추가해서 넣어두었어요!

cacaocoffee · 2024-06-14T22:14:14Z

app/src/main/java/com/sopt/now/ui/login/LoginViewModel.kt

+import androidx.lifecycle.viewModelScope
 import com.sopt.now.data.model.User
-import com.sopt.now.util.StringNetworkError.FAIL_ERROR
-import com.sopt.now.util.StringNetworkError.LOGIN
-import com.sopt.now.util.StringNetworkError.SERVER_ERROR
+import com.sopt.now.domain.entity.request.AuthRequestModel
+import com.sopt.now.domain.usecase.LogInUseCase


너무 잘못쓰고있었네요..
말씀해주신 주제와 아키텍처 여러 관련된 자료 찾아보고 공부한다고 답변을 너무 늦게 달았네요

제대로 한지는 모르지만 clean 아키텍처에 대해서 찾아 공부하고 적용을 하려 노력을 했습니다.

앱잼에 제대로 들어가전에 구글권장아키텍처도 제대로 확인해보고 가려구요 감사합니다.

cacaocoffee and others added 11 commits June 4, 2024 02:29

[mod] #18 SignUpViewModel 예외처리 변경

c3591ce

[mod] #18 api login 변경

7e3a2b4

[mod] #18 api getUsrInfo 변경

e9bb79d

[mod] #18 datasource -> dto 로 변경

ae881b6

[chore] #18 코드정리

23151c3

[add] #18 repository 생성

b41a135

[add] #18 LoginViewModelFactory 추가

f1ad914

[mod] #18 Login 레포지토리로 로직 변경

8339904

[mod] #18 signup repository 패턴적용

ad8381d

[mod] #18 signup usecase 분리

4784add

[mod] #18 login usecase 분리

adb8543

cacaocoffee added the 📕필수과제_xml label Jun 7, 2024

cacaocoffee requested a review from a team June 7, 2024 03:45

cacaocoffee self-assigned this Jun 7, 2024

cacaocoffee requested review from OliviaYJH, kez-lab, chanubc and hyoeunjoo and removed request for a team June 7, 2024 03:45

cacaocoffee changed the base branch from main-xml to develop-xml June 7, 2024 03:51

cacaocoffee added 3 commits June 7, 2024 13:10

[chore] #18 data <-domain 의존성 분리를 위해 dto와 로직변경

6029ae8

[chore] #18 data <-domain 의존성 분리를 위해 dto와 로직변경

2cc557b

Merge remote-tracking branch 'origin/feat/week-xml-07' into feat/week…

1c197b0

…-xml-07

librarywon reviewed Jun 9, 2024

View reviewed changes

hyoeunjoo reviewed Jun 9, 2024

View reviewed changes

OliviaYJH approved these changes Jun 9, 2024

View reviewed changes

chanubc reviewed Jun 9, 2024

View reviewed changes

cacaocoffee added 3 commits June 12, 2024 23:54

[mod] #18 usecase 변경

3e6e6ba

[mod] #18 mapper 추가 의존성변경

d752e67

[Chore] #18 user 폴더 Ui로 변경

05219f1

cacaocoffee added 9 commits June 13, 2024 06:22

[Chore] #18 ViewModelFactory 하나로 통합

29a96db

[Mod] #18 AuthService 수정

381ecab

[Add] #18 getErrorMessage,ApiError ,BaseResponseEntity 추가

f639c9a

[Mod] #18 코드리뷰반영

06dbdef

[Mod] #18 AuthRequestModel ->UserEntity 이름변경

f4bb9c1

[Feat] #18 getinfo domain과 data 영역 추가 및 수정

32a8c17

[Feat] #18 getinfo repository 패턴 적용

d91ec8a

[Chore] #18 코드 정리

94ace2d

[Chore] #18 코드 리뷰 반영 viewmodel 타입명시

46038a3

cacaocoffee commented Jun 14, 2024

View reviewed changes

cacaocoffee added 7 commits June 19, 2024 16:17

[Add] #18 hilt gradle 추가

a2f41df

[Add] #18 HiltAndroidApp 적용 app 추가

60b29dc

[Add] #18 Api, DataModule 추가

0c69554

[Mod] #18 hilt 적용

b374cf0

[Del] #18 hilt 적용 후 안쓰는 파일 제거

df0c370

[Mod] #18 ApiModule 위치 변경

e2a65aa

[Mod] #18 kapt -> ksp

256d629

		private val _loginState = MutableLiveData<UiState<User>>()
		val loginState = _loginState

Feat/week xml 07 #19

Are you sure you want to change the base?

Feat/week xml 07 #19

Conversation

cacaocoffee commented Jun 7, 2024

🛠 Related issue

✏️ Work Description

📸 Screenshot

😅 Uncompleted Tasks

📢 To Reviewers

✅ Check List

librarywon left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyoeunjoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

OliviaYJH left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

chanubc left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cacaocoffee left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

librarywon left a comment •

edited

Loading