Skip to content

디스코드 봇 교체#42

Merged
DaleSeo merged 1 commit into
mainfrom
refactor/unify-discord-credentials
Jun 28, 2026
Merged

디스코드 봇 교체#42
DaleSeo merged 1 commit into
mainfrom
refactor/unify-discord-credentials

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jun 28, 2026

Copy link
Copy Markdown
Member

디스코드 봇을 개인 계정에서 커뮤니티 공용 DaleStudy 봇으로 일원화하는 작업입니다.

DISCORD_BOT_TOKEN: ${{ secrets.DISCORD_BOT_TOKEN }}
DISCORD_TOKEN: ${{ secrets.DISCORD_TOKEN }}
DISCORD_GUILD_ID: ${{ secrets.DISCORD_GUILD_ID }}
DISCORD_ROLE_ID: ${{ secrets.DISCORD_ROLE_ID }}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

얘는 쓰이는데가 없어서 그냥 삭제하였습니다.

Comment on lines +47 to +49
GITHUB_APP_ID
GITHUB_APP_INSTALLATION_ID
GITHUB_APP_PRIVATE_KEY

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

무슨 앱인지 너무 햇갈려서 앞에 GITHUB_ 접두어를 붙였습니다.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

저도 처음에 github를 넣었었는데 github를 뺀게 github를 빼면 워크플로우 어딘가에서 보안 관련 이슈로 막았던 걸로 기억합니다!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

이 부분 맥락을 기록해놓지 않아서 지금 어디인지 확실하지 않네요! 우선 넣어보고 문제가 보이면 대처해야할 것 같습니다!

Comment thread README.md
| Key | 설명 |
| ------------------ | ----------------- |
| `GITHUB_ORG` | GitHub 조직 이름 |
| `GITHUB_TEAM_SLUG` | 초대할 팀 slug |

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

요거는 outdated 된 거 같아서 함께 수정했습니다.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

넵 저거 outdated 된게 맞네요! 감사합니다!

@DaleSeo DaleSeo merged commit 2c57335 into main Jun 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants