Skip to content

Conversation

@chysis
Copy link
Collaborator

@chysis chysis commented Feb 2, 2025

이슈번호

요약(개요)

  • PR 리뷰 시 생성한 사람이 멘션되지 않는 버그를 수정했습니다.
    • id가 아닌 name으로 매핑하고 있기 때문에 문제가 발생했습니다.
  • 프론트엔드 이슈 템플릿 관련해서 사소한 수정 사항이 있습니다. 별도의 이슈를 만들 만큼의 일이 아니기 때문에 setting PR에서 함께 작업했습니다.

작업 내용

  • PR 리뷰 시 생성한 사람이 멘션되지 않는 버그 수정
  • 프론트엔드 이슈 템플릿 잘못 생성한 파일 삭제 및 불필요한 줄바꿈 제거

집중해서 리뷰해야 하는 부분

기타 전달 사항 및 참고 자료(선택)

@chysis chysis added setting setting ALL Frontend + Backend + Ios 철민 labels Feb 2, 2025
@chysis chysis self-assigned this Feb 2, 2025
@chysis chysis linked an issue Feb 2, 2025 that may be closed by this pull request
1 task
Copy link
Member

@zelkovaria zelkovaria left a comment

Choose a reason for hiding this comment

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

Discord 알림 설정해주시느라 고생하셨어요! 방금 디스코드 알림에 @everyone 처럼 뜨는 부분을 수정하신 거로 이해했는데 맞을까요! 맞다면 덕분에 편한 작업을 할 것 같아요 감사합니다 ✨

@chysis
Copy link
Collaborator Author

chysis commented Feb 2, 2025

image
사진처럼 PR에 리뷰하면 PR 주인을 멘션해야 하는데, 아이디 매핑이 제대로 되지 않아서 <@>로 출력되고 있어요! 이 부분을 수정했습니다. 이제 PR 주인에게 멘션이 제대로 갈 거예요.

새로운 PR이 등록됐을 때 @everyone으로 모두를 태그하는 것은 그대로 유지됩니다 :)

@chysis chysis merged commit b3783b5 into main Feb 2, 2025
2 checks passed
@chysis chysis deleted the all/setting/16-discord-mention-bug branch February 2, 2025 13:02
onegood07 pushed a commit that referenced this pull request Feb 28, 2025
…ion-bug

[ALL] setting: 디스코드 알림 오류 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ALL Frontend + Backend + Ios setting setting 철민

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ALL] setting: 디스코드 알림 오류 수정

3 participants