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

[feature] 쥬스레터 발송 API #56

Open
2 of 3 tasks
Haeun-Y opened this issue Sep 26, 2023 · 3 comments
Open
2 of 3 tasks

[feature] 쥬스레터 발송 API #56

Haeun-Y opened this issue Sep 26, 2023 · 3 comments
Assignees
Labels
✨ feature New feature

Comments

@Haeun-Y
Copy link
Collaborator

Haeun-Y commented Sep 26, 2023

📌 구현 기능

쥬스레터 API

🛠 작업 내용

  • 메일 발송 API
  • html 템플릿 수정
  • 구독 등록/취소 API

📢 의논 사항

  • 사용자 이메일이 null인 걸로 구독여부를 체크하는 것이 괜찮을지 고민이 됩니다. 의견 부탁드립니다. user의 email column 값이 null일 때 구독하지 않았다고 구분하는 것이 null 데이터의 의미에 맞지 않아 위험할 것 같아서요.
@Haeun-Y
Copy link
Collaborator Author

Haeun-Y commented Sep 29, 2023

@JoongHyun-Kim 의논사항 업데이트했습니다. 확인 부탁드립니다.

@JoongHyun-Kim
Copy link
Member

@JoongHyun-Kim 의논사항 업데이트했습니다. 확인 부탁드립니다.

음 혹시 위험하다는 게 어떤 의미인가요? 로직적으로만 생각하면 null로 구분해도 괜찮을 것 같은데, 구독여부 컬럼을 따로 만들어도 될 것 같긴 합니다!

@Haeun-Y
Copy link
Collaborator Author

Haeun-Y commented Sep 29, 2023

null값인지 아닌지를 어떤 로직의 기준으로 삼으면 안좋다는 얘기를 주워들은 것 같아서요. 이메일 주소가 null인 것과 구독 여부를 null값인지를 확인해서 판단하는건 데이터 관리 측면에서 혼동을 초래해 안좋을 것 같아서요. 그럼 일단은 null로 구분하고 주워들은 것의 출처를 명확히 확인해본 다음 다시 말씀드릴게요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature New feature
Projects
None yet
Development

No branches or pull requests

2 participants