Conversation
|
알림 기능 문제 없어보입니다! |
| List<Purchase> purchases = purchaseRepository.findAllByEventId(eventId); | ||
| log.debug(">> 이벤트 구매자 조회 완료: eventId={}, 구매자 수={}", eventId, purchases.size()); | ||
|
|
||
| // 모든 구매자에게 행사 취소 알림 전송 |
There was a problem hiding this comment.
행사 취소 이전, 구매한 티켓이 존재한다면 해당 티켓들에 대한 취소가 우선적으로 이루어져야 할 듯 합니다.
취소와 티켓 환불을 함께 고려한 로직은 구현되지 않아, 해당 로직은 리펙토링 진행 시 반영하도록 하겠습니다.
There was a problem hiding this comment.
확인했습니다. 해당 로직 구현 후 알림 주시면 관리자 대시보드에도 반영하겠습니다.
추가 첨언 없으시면 반응 달아주세요.
|
|
||
| try { | ||
| // 삭제된 이벤트 조회 | ||
| List<Event> events = eventRepository.findAllByIsDeletedTrue(); |
There was a problem hiding this comment.
삭제 대기 중인 이벤트 목록 조회 -> isDeleted 값이 true 인 이벤트의 목록을 조회하는게 맞을까요?
삭제 대기 중 이라는 말이 조금 어색하게 보여지는 듯 합니다...!
There was a problem hiding this comment.
현재 관리자 대시보의 이벤트 관리 페이지를 보면 이벤트 목록 조회 탭과 삭제된 이벤트 탭으로 나뉘어 있습니다. isDelete값이 true인 이벤트의 경우 삭제된 이벤트 탭으로 이동하게 되고, 여기서 복구 버튼을 누르면 다시 이벤트 목록 조회 탭으로 되돌아오게 됩니다. 해서, 위와 같이 구현하였습니다.
때문에 isDeleted가 true 값의 경우 삭제된 이벤트에서 조회되기 때문에 문제가 없을 것으로 보이는데 다른 의견 있으실까요?
There was a problem hiding this comment.
삭제 되어 대기중이라는 의미라는 말씀이신거죠??
삭제 대기 중이라는 의미를 저는 "삭제 요청 -> 삭제 대기 -> 삭제 승인 -> 최종 삭제" 라는 플로우로 느껴서 어색했던 것 같습니다!
복구 가능하다 라는 의미에서의 "삭제 대기 중" 이라면 합당해 보입니다!
There was a problem hiding this comment.
맞습니다. 추가로, 삭제된 이벤트 탭에서도 완전히 지울 수 있도록하는 기능을 추가해야 겠네요.
재확인차 질문합니다만 DB에서는 관련 데이터는 어떤 경우에서도 지워지지 않도록 하는게 맞는건지 확인부탁드립니다.
There was a problem hiding this comment.
맞습니다. DB 에서는 한번 저장된 이벤트는 실제 데이터 삭제가 이루어 지지 않는 것이 맞습니다.
🔎 작업 내용
관리자 대시보드
✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.