-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FEAT] 진행 상황 알림 Discord 봇 구현 #454
base: develop
Are you sure you want to change the base?
Conversation
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에 대해 오늘 날짜의 커밋 개수를 보여주는 게 작업 현황 공유에 큰 효용이 있을지는 살짝 모르겠어서 자세히 설명해주시면 좋을 것 같아요!
|
||
async function getCommits(owner, repo, branch) { | ||
try { | ||
const now = new Date(); |
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.
💭 질문 💭
한국 기준으로 계산되는 건가요?
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.
시간대 한국 기준으로 수정하였습니다. !
.github/workflows/progress-alarm.yml
Outdated
owner, | ||
repo, | ||
sha: branch, | ||
since: startOfYesterday.toISOString(), |
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.
💭 질문 💭
toISOString는 utc 기준이여서 한국 기준으로 계산되어야 할 것 같아요!
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.
자정 시간을 잘못 계산했네요 ㅠ
한국 기준에 맞게 수정하였습니다!
|
||
for (const pr of pullRequests.data) { | ||
const branch = pr.head.ref; | ||
const commits = await getCommits(owner, repo, branch); |
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의 커밋 중 오늘 날짜의 커밋만 가져오는 건가요?
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.
오전 7시에 전날 24시간 동안 작업된 commit들을 가져오는 방식입니다!
네, 피드백 확인하고 수정사항 반영하였습니다! commit 수는 처음에는 commit 메세지를 정리해서 보여주면 어떤 작업을 했는지 구체적으로 알 수 있을거라 생각했습니다. |
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이 올라와서 리뷰 요청을 해야 알 수 있는데 (백엔드는 그 알림도 없기도 하고) 이제는 PR 올리기 전 작업 중에도 알 수 있겠네요 ~ !
꼭 반영하지 않아도 되지만 이슈 제목에다 해당 이슈의 바로가기 링크를 넣는 방법도 좋을 것 같아요! 아무래도 어떤 의도로 이 작업을 하고 있는지 자세하게 컨공이 안 된 경우가 있을 수 있으니 !!
필요한 부분이 무엇인지 고민하고 척척 만들어 낸 포메 칭찬해 ~~~ 수고했어요 🤗🤗
Issue Number
#443
As-Is
프로젝트가 비대면으로 전환된 상태에서 서로의 진행 상황을 알기 어렵기 때문에 각자가 하루 동안 올린 commit 들을 모아서
요약해서 메세지를 보내주는 Discord 봇 구현
To-Be
Check List
Test Screenshot
(Optional) Additional Description
🌸 Storybook 배포 주소