kijin
commented
2 months ago https://github.com/rhymix/rhymix/pull/2401#issuecomment-2323133077
여러 회원에게 각각 다른 데이터를 전송하고 싶다면 각각 다른 Push 인스턴스를 생성하시기 바랍니다.
이미 push.send (before) 트리거가 있으니, 위의 권고사항만 철저하게 지켜 주신다면 별도의 트리거는 필요하지 않을 것으로 사료됩니다. 코어에서 생성하는 푸시 알림은 모두 개인별로 발송되므로 문제가 없고, 서드파티 자료는 만들기 나름입니다.
관리자가 여러 글을 동시에 이동시키는 등 불가피한 상황이 아닌 이상, 루프 안에서 트리거를 호출하는 것은 잘못된 설계입니다. 안 그래도 느리다고 #2402 이슈를 등록하셨는데, 상황을 더 악화시키면 어떻게 합니까?
게다가 회원마다 URL에 member_srl을 붙이지 않더라도 누가 클릭했는지 추적할 방법은 얼마든지 있을 텐데, 이런 꼼수를 이유로 코어의 구조를 비틀어놓는 것은 더이상 라이믹스에서는 받아들이지 않겠습니다.
위의 권고사항을 따르지 않을 목적으로 동일한 취지의 PR을 다시 등록하실 경우, 그것도 거부하겠습니다.
앞으로의 성능 개선은 개인별로 Push 인스턴스를 생성한다는 가정 하에 진행하겠습니다. 하나의 인스턴스를 이리저리 던져가며 복잡하게 처리하기보다는, 단순한 작업을 반복하기 쉽도록 설계하는 것이 라이믹스 유저들에게는 더 편리할 것 같습니다.
앞의 pr 과 같은 목적이나 구현은 트리거로 했습니다. 메세지 다르게 해도 비동기 전송 가능하게 서드파티에서 조절할 수 있을 것입니다.