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

[Feat/#131] 이메일 구독 내용 채워서 구현 #156

Merged
merged 11 commits into from
Jul 8, 2024

Conversation

belljun3395
Copy link
Collaborator

🎫 연관 이슈

resolved: #131

💁‍♂️ PR 내용

  • 이메일 구독 내용 채워서 구현

🙏 작업

  • SendArticleEmailArgs 구체화
  • BatchCategoryType, BatchMemberType 추가: data 의존성을 추가하는 것 보다 동일한 이넘을 만들고 javadocs로 관련 기록을 하는 것이 좋지 않을까 판다하여 추가
  • 이메일 내용을 채우기 위한 조회 추가: 기존의 id로만 가지고 있던 것을 활용하여 이메일 내용에 필요한 정보 조회 추가 (CONTENT, TITLE, CATEGORY_CD, WRITER NAME, URL)

🙈 PR 참고 사항

  • email 탬플릿 수정 사항 클라이언트에 전달 받아 추가해야함

📸 스크린샷

🤖 테스트 체크리스트

  • 체크 미완료
  • 체크 완료

@belljun3395 belljun3395 requested a review from hun-ca as a code owner July 4, 2024 13:29
@github-actions github-actions bot added the feature 새로운 기능을 만들 때 사용됩니다 label Jul 4, 2024
Copy link
Member

@hun-ca hun-ca left a comment

Choose a reason for hiding this comment

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

코드 성 enum 클래스 모듈별 중복 발생 해결 이슈 #168 는 따로 등록해놨습니다.
나중에 리펙토링하면서 해결하는걸로 하시죠

Comment on lines +7 to +27
enum class BatchCategoryType(val code: Byte, val displayName: String) {
ECONOMY(0, "경제"),
IT(10, "IT"),
MARKETING(20, "마케팅"),
CULTURE(30, "교양"),
SCIENCE(40, "과학");

companion object {
fun fromCode(code: Byte): BatchCategoryType? {
return entries.find { it.code == code }
}

fun convertToCode(displayName: String): Byte {
return entries.find { it.name == displayName }?.code ?: throw IllegalArgumentException("Invalid category name")
}

fun convertToDisplayName(code: Byte): String {
return entries.find { it.code == code }?.displayName ?: throw IllegalArgumentException("Invalid category code")
}
}
}
Copy link
Member

Choose a reason for hiding this comment

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

CategoryType과 BatchCategoryType이 가진 카테고리 값들은 왜 서로 다른가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@belljun3395 belljun3395 merged commit 17ee380 into main Jul 8, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새로운 기능을 만들 때 사용됩니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이메일 구독 내용 채워서 구현
2 participants