구글 태그 및 채널톡 버튼 추가, 코드리뷰 마감시간 알림 슬랙봇 구현 #496
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
review-request |
|
review-request |
Gilpop8663
reviewed
Sep 1, 2023
| })(); | ||
|
|
||
| ChannelIO('boot', { | ||
| pluginKey: '1d876858-22cd-444b-8fcd-33487aed8247', |
There was a problem hiding this comment.
플러그인 키는 숨기지 않아도 괜찮은 키인가요?
| <!-- Google Tag Manager (noscript) --> | ||
| <noscript | ||
| ><iframe | ||
| src="https://www.googletagmanager.com/ns.html?id=GTM-N7NTHNF7" |
| const nextDayHour = nextDay.getUTCHours(); | ||
| const nextDayDate = nextDay.getUTCDate(); | ||
| const nextDayMonth = nextDay.getUTCMonth() + 1; | ||
| console.log(nextDayHour, nextDayDate, nextDayMonth); |
Comment on lines
19
to
22
| } else if (prCreatedHour === 22 || prCreatedHour === 23) { | ||
| deadline = `내일(${nextDayMonth}월 ${nextDayDate}일) 20시 00분`; | ||
| } else if (prCreatedHour + 10 >= 22) { | ||
| deadline = `내일(${nextDayMonth}월 ${nextDayDate}일) ${ |
There was a problem hiding this comment.
조건문의 조건의 가독성을 올려보시는 것을 제안해봅니다 😀
else if (prCreatedHour>=22) {
deadline = `내일(${nextDayMonth}월 ${nextDayDate}일) 20시 00분`;
} else if (prCreatedHour >= 12) {
deadline = `내일(${nextDayMonth}월 ${nextDayDate}일) ${
chsua
approved these changes
Sep 1, 2023
| "type": "section", | ||
| "text": { | ||
| "type": "mrkdwn", | ||
| "text": "리뷰 완료했습니다👍\n<${{ github.event.comment.html_url }}|리뷰어의 코멘트 확인하러 가기>" |
There was a problem hiding this comment.
pr 외에도 pr에 리뷰가 달린 것도 알림이 오나요??
| console.log(nextDayHour, nextDayDate, nextDayMonth); | ||
|
|
||
| if (prCreatedHour < 10 && prCreatedHour > 0) { | ||
| deadline = `오늘(${prCreatedMonth}월 ${prCreatedDate}일) 20시 00분`; |
There was a problem hiding this comment.
그냥 궁금한건데 이 deadline을 선언하는 곳이 없는데 어떻게 사용 가능한가요??
| deadline = `오늘(${prCreatedMonth}월 ${prCreatedDate}일) ${ | ||
| prCreatedHour + 10 | ||
| }시 ${prCreatedMinute}분`; | ||
| } |
There was a problem hiding this comment.
혹시 위 조건이 잘 이해가 안되는데ㅠㅠ
정리된 경우의 수가
- 오늘 안에 되는 경우
- 자정(00시) 이후부터의 PR
- 00시 ~ 10시 PR: 당일 20시
- 10시 ~ 12시 PR: 기존 시간 + 10시간
- 자정(00시) 이후부터의 PR
- 다음날로 넘어가는 경우
- 정오(12시) 이후부터의 PR
- 12시 ~ 22시 PR: 기존 시간 + 12시간(22시~10시까지의 시간) + 10시간
- 22시 ~ 00시 PR: 다음날 20시
- 정오(12시) 이후부터의 PR
이렇게 네 가지 케이스로 볼 수 있을 것 같아요!
여기서 오늘/내일이란 말을 없애고 그냥 월/일/요일로 작성하면 더 가독성있고, 코드 경우의 수도 줄어들 것 같은데 어떠신가요??
There was a problem hiding this comment.
생각해보니까 오늘/내일 때문에 더 복잡해진거였네요😅😅
제안 감사합니다! 반영해서 새로운 pr로 다시 올릴게요~
(새로운 pr을 다시 올려야 github actions이 제대로 작동하는지 확인해볼 수 있어서..😂)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥 연관 이슈
close: #492
close: #494
📝 작업 요약
index.html에 추가하였습니다.⏰ 소요 시간
🔎 작업 상세 설명
votogether.com사이트의 script에 구글 태그 관련 코드를 추가해야 했습니다.🌟 논의 사항