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] 마이페이지 알림 수신 여부 API #165

Merged
merged 2 commits into from
Sep 6, 2023

Conversation

chaerlo127
Copy link
Collaborator

🌱 이슈 번호

close #160


💬 기타 사항

  • valid -> boolean이 아니라 다른 타입이어도 에러를 잡지 못하는 오류 해결 필요
  • 약간 고민인게 boolean 을 req로 보내는데 이게 필요한지 궁금함니다
  • 값을 보내고 값을 변경시킨후에 변경된 값을 res로 보내는게 나을까욤

@chaerlo127 chaerlo127 added 🔮 feat 기능을 생성해요 ! 🐣 chae-eun 채은이가 만든 기능이에요 ! 👩🏻‍💻 users 사용자 도메인 기능이에요 ! 🔔 notifications 알림 기능이에요 ! labels Sep 5, 2023
@chaerlo127 chaerlo127 self-assigned this Sep 5, 2023
Copy link
Member

@sojungpp sojungpp left a comment

Choose a reason for hiding this comment

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

💬 기타 사항

  • valid -> boolean이 아니라 다른 타입이어도 에러를 잡지 못하는 오류 해결 필요
  • 약간 고민인게 boolean 을 req로 보내는데 이게 필요한지 궁금함니다
  • 값을 보내고 값을 변경시킨후에 변경된 값을 res로 보내는게 나을까욤

음 저는 쇼핑 쪽에서 게시글 좋아요 구현했을 때 따로 request body 받지 않고
요청 들어오면 현재 boolean 값과 반대로 변경하도록 했는데 그건 어때여?

/**
* 마이페이지 알림 수신 여부
*/
@PostMapping("/noti")
Copy link
Member

Choose a reason for hiding this comment

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

url은 조금 길어져도 명확하게 단어 전체 적어주는 거 어때용 /notification

@chaerlo127
Copy link
Collaborator Author

음 저는 쇼핑 쪽에서 게시글 좋아요 구현했을 때 따로 request body 받지 않고 요청 들어오면 현재 boolean 값과 반대로 변경하도록 했는데 그건 어때여?

저도 그게 간단할거 같아요 좋슴니당!!

@chaerlo127 chaerlo127 merged commit e0e42f1 into develop Sep 6, 2023
1 check passed
@sojungpp sojungpp deleted the feat/#160-postNotiStatus branch October 21, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐣 chae-eun 채은이가 만든 기능이에요 ! 🔮 feat 기능을 생성해요 ! 🔔 notifications 알림 기능이에요 ! 👩🏻‍💻 users 사용자 도메인 기능이에요 !
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 마이페이지 알림 수신 여부 수정 API
3 participants