-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIX] 디스코드 알림 기능 수정 #37
Conversation
- 디스코드 알림 이미지 수정 - 디스코드 알림 이름 수정 - 디스코드 알림 임베드 제목, 임베드 내용 맞변경
Test Results12 tests 12 ✅ 0s ⏱️ Results for commit 6e3ff80. |
📝 Test Coverage Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/noti
커찬, 말씀해주신 discsord 웹 훅 개선사안 확인했습니다! comment 들은 주로 제가 이해한 내용이 맞는지 확인하는 내용이라 크게 신경쓰지 않으셔도 될 것 같습니다~
고생많으셨어요!
@@ -24,8 +22,8 @@ jobs: | |||
echo "PR_URL=${{ github.event.issue.pull_request.html_url }}" >> $GITHUB_ENV | |||
echo "AUTHOR_URL=${{ github.event.comment.user.avatar_url }}" >> $GITHUB_ENV | |||
echo "CONTENT=$(echo '${{ github.event.comment.body }}' | base64 -w 0)" >> $GITHUB_ENV | |||
echo "COMMENT_BODY=${{ github.event.issue.title }}" >> $GITHUB_ENV | |||
echo "TITLE=Pull Request" >> $GITHUB_ENV | |||
echo "TITLE=${{ github.event.issue.title }}" >> $GITHUB_ENV |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 이슈 & PR 제목을 title로 하고 comment_body로 구분해주는 거군요!
username: $username, | ||
avatar_url: $avatar_url, | ||
username: $author_name, | ||
avatar_url: $author_url, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ $author_url은 작성자 프로필 이미지를 가져오는 변수명으로 생각되네요! 좋아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/noti
크게 리뷰할 내용이 없어 바로 approve합니다~
@@ -24,8 +22,8 @@ jobs: | |||
echo "PR_URL=${{ github.event.issue.pull_request.html_url }}" >> $GITHUB_ENV | |||
echo "AUTHOR_URL=${{ github.event.comment.user.avatar_url }}" >> $GITHUB_ENV | |||
echo "CONTENT=$(echo '${{ github.event.comment.body }}' | base64 -w 0)" >> $GITHUB_ENV | |||
echo "COMMENT_BODY=${{ github.event.issue.title }}" >> $GITHUB_ENV |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
🚩 연관 이슈
closed #36
🗣️ 리뷰 요구사항 (선택)