Skip to content

Conversation

@ThirFir
Copy link
Member

@ThirFir ThirFir commented Jan 25, 2026

이슈

내용

image

위 API들에 대한 데이터 레이어 구조를 구현했습니다 (Repository - DataSource - Retrofit)

[또 1000줄을 넘겨버렸는데... 대부분 단순 중복이라 훅훅 넘겨주세요 ㅠ]

ThirFir and others added 30 commits January 25, 2026 12:33
- Json provide 메서드
- ConverterFactory 부착
- CallAdapterFactory 부착
- 매핑 로직
- Experience 도메인 모델
- Hilt 모듈
@ThirFir ThirFir requested a review from Hyesung82 January 25, 2026 04:45
@ThirFir ThirFir self-assigned this Jan 25, 2026
internal interface RepositoryModule {

@Binds
@ActivityScoped

Choose a reason for hiding this comment

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

ActivityRetainedComponent는 화면 회전 시에도 유지되게 하는 설정값이고 ActivityScoped는 무조건 Activity 생명주기에 따라 소멸하는 설정값인 것 같은데(잘 모름)
이거 혹시 @ActivityRetainedScoped를 써야 의도대로 동작하지 않을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

앗 이런!! 수정할게여


fun startChattingStream(questionId: String, sendingMessage: String): Flow<ChattingStreaming>
suspend fun getChatHistory(questionId: String): Result<ChattingHistory>
suspend fun updateLetter(chattingId: String, questionId: String, content: String): Result<Unit>

Choose a reason for hiding this comment

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

혹시 이게 자소서 업데이트하는 함수라면 updatePaper로 이름 지으면 어떨까요?
왜냐하면 Figma에서 자소서 아이콘 이름이 paper라서.. ㅎㅎ
아니면 아예 cover letter라고 표현하는 것도 괜찮을 것 같습니다

Copy link
Member Author

Choose a reason for hiding this comment

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

이게 백엔드에서는 Draft로 표현하는거 같더라구요 ...
일단 다같이 얘기를 해봐야할듯 합니다 !!

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