discordjs 버전 업데이트

[chshin59]

Why?

• discord.js 업데이트에 따라 코드 변경이 필요해짐
• 장점
  • Discord Developer Portal에서 제공하기 시작한 API 문서를 기반으로 API를 맞추고 있는 느낌입니다.
  • REST API의 분리로 gateway와 상관없이 API 호출 가능
  • typescript로 작성되어 타입을 참고하기 편해짐

How?

• package.json의 discord 의존성 변경 "discord.js": "^14.7.1" -> "@discordjs/core": "^0.6.0"
• example code 참고

고민거리

• 사용자가 음성채널에 입장, 퇴장할 때의 이벤트인 voiceStateUpdate가 oldVoiceState, newVoiceState를 둘 다 제공해주었는데 지금은 newVoiceState하나만 제공해줍니다. (아마도 기존에는 voiceState 캐싱하는 내부 코드가 있어서 가능했었던 걸로 보입니다.)

  Study Session에 사용자의 voiceState를 추가로 저장해서 구현할까 싶습니다. (studySession.voiceState 가 존재하지 않거나 음성채널에 없거나 라운지 채널에 있는 경우 vs 아닌 경우 로 나눠서 기존처럼 구현)

  StudySession {
  pageId: string;
  startTime: Date;
  endTime?: Date;
  isEarlyEnded?: boolean;
  voiceState?: GatewayVoiceState <-
  }

다른 의견은 언제나 환영입니다!

[smj53]

Study Session에 사용자의 voiceState를 추가로 저장해서 구현할까 싶습니다.

User에 넣는 건 어떨까요? User의 상태이니 User가 voiceState를 갖고 있는 게 개념적으로도 맞는 것 같고, StudySession의 생성/삭제가 잦아서 처리가 좀 복잡해질 것 같습니다. 물론 StudySession 처리 로직이 이상하다면 그 쪽을 바꾸고 StudySession에 넣어도 좋습니다!

+지금보니 StudySession의 삭제는 오류가 아닌 이상 발생하지 않는군요. 휴식 기능에서도 공부방 음성채널 접속 여부가 필요해서 StudySession에 넣는 것도 나쁘지 않을 것 같네요.