Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

📄 요약(Summary)

알림 한 개에 대해 읽음 처리 기능 구현

✍🏼 상세(More)

PR Desciption

변경 사항 설명

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

  • 현재 기획 flow가 알림 목록에서 하나 클릭했을 때 알림에 해당하는 작업 상세 조회 페이지로 이동합니다.
  • 이에 따라 알림 한 개 읽음 처리 API에서 return 값으로 해당 작업 상세 조회 값을 반환해서 front가 API를 두 번 사용하지 않도록 하는 것이 어떤가요?

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • [ ✅ ] PR 양식에 맞게 작성했습니다
  • [ ✅ ] 모든 테스트가 통과했습니다
  • [ ✅ ] 프로그램이 정상적으로 작동합니다
  • [ ✅ ] 적절한 PR 라벨을 설정했습니다
  • [ ✅ ] 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

@starboxxxx starboxxxx requested a review from joowojr January 25, 2025 17:24
@starboxxxx starboxxxx self-assigned this Jan 25, 2025
@starboxxxx starboxxxx added ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 labels Jan 25, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

Upadate 보다는 Readnotification이 명확할거같습니다!

@Transactional
@Override
public void updateNotification(Long notificationId) {
Notification notification = loadNotificationPort.findById(notificationId).get();
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 optional로 받아올 경우 예외 처리 하셔야될거같아요!

Copy link
Contributor

@joowojr joowojr left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 리뷰 확인부탁드립니다!

@joowojr joowojr changed the title CLAP-136 Feature: 알림 한 개 읽음 처리 기능 구현 CLAP-136 알림 한 개 읽음 처리 기능 구현 Jan 26, 2025
@starboxxxx starboxxxx merged commit 320aac6 into develop Jan 26, 2025
1 check passed
@starboxxxx starboxxxx deleted the CLAP-136 branch January 26, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants