Skip to content
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

Merged
merged 1 commit into from
Dec 27, 2024
Merged

[FIX] 디스코드 알림 기능 수정 #37

merged 1 commit into from
Dec 27, 2024

Conversation

leegwichan
Copy link
Contributor

🚩 연관 이슈

closed #36

🗣️ 리뷰 요구사항 (선택)

- 디스코드 알림 이미지 수정
- 디스코드 알림 이름 수정
- 디스코드 알림 임베드 제목, 임베드 내용 맞변경
@unifolio0 unifolio0 added the fix 기능 수정 label Dec 27, 2024
@unifolio0 unifolio0 self-requested a review December 27, 2024 08:06
Copy link

Test Results

12 tests   12 ✅  0s ⏱️
 4 suites   0 💤
 4 files     0 ❌

Results for commit 6e3ff80.

Copy link

📝 Test Coverage Report

Overall Project 79.02%

There is no coverage information present for the Files changed

Copy link
Contributor

@coli-geonwoo coli-geonwoo left a 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
Copy link
Contributor

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,
Copy link
Contributor

Choose a reason for hiding this comment

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

✅ $author_url은 작성자 프로필 이미지를 가져오는 변수명으로 생각되네요! 좋아요.

Copy link
Contributor

@unifolio0 unifolio0 left a 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
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@leegwichan leegwichan merged commit b6bf707 into develop Dec 27, 2024
6 checks passed
@leegwichan leegwichan deleted the fix/#36 branch December 27, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 디스코드 알림 기능 수정
3 participants