Skip to content

Conversation

@choijungp
Copy link
Collaborator

🌁 Background

WhiteboardObjectUseCase init시 프로필을 가져오니까 프로필을 바꾼 후에도 이전 프로필을 가져오는 이슈가 있었습니다.
init시에는 profileRepository만 받고 profile 사용 시에 load할 수 있도록 변경하였습니다.

📱 Screenshot

로직 변경이라 한 기기 테스트 영상만 첨부하겠습니다.
Simulator Screen Recording - iPhone SE (3rd generation) - 2024-12-02 at 17 49 42

👩‍💻 Contents

myProfile이 필요한 함수마다 profileRepository에서 profile을 load해서 사용하도록 수정하였습니다.

✅ Testing

화이트보드에서 오브젝트 선택 시 변경된 프로필 이모지로 잘 표시되는지 확인해보세요 !

📝 Review Note

프로필을 load하는 것이 반복된다고 생각이 드는데 ... 그렇다고 초기에 profile를 load 해놓을 순 없어 일단 이렇게 했습니다 .. .. .
더 나은 방법이 있다면 언제든 웰끔

@choijungp choijungp self-assigned this Dec 2, 2024
Copy link
Collaborator

@taipaise taipaise left a comment

Choose a reason for hiding this comment

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

수정해주셔서 감사합니다~!

Copy link
Member

@eemdeeks eemdeeks left a comment

Choose a reason for hiding this comment

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

고생 하셨습니다!!

매 함수에서 load를 불러오는 동작이 진행 될 것 같은데요,
그러지 않기 위해선 밖에서 Profile을 주입받아야 할 것 같습니다.
하지만 현재 Usecase를 선언해주는 곳이 SceneDelegate에서 한번 생성한 Usecase를 사용하다보니 쉽지 않은 것 같네요...
현재 상황에서는 현실적인 해결 방안인것 같습니다!! 구욷

나중에 다같이 의존성 주입에 대해서 고민 해 봅시다 :)

Copy link
Collaborator

@ekrud99 ekrud99 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 조이~ 👏👏👏

@choijungp choijungp merged commit 62b6572 into develop Dec 2, 2024
2 checks passed
@choijungp choijungp deleted the fix/ObjectSelect branch December 2, 2024 10:56
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.

5 participants