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.

Sign up for GitHub

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

Jump to bottom

(WIP) 웹훅을 사용하여 이쁘게 메세지 전송하기 구현 #10

Closed
wants to merge 4 commits into from
Closed

(WIP) 웹훅을 사용하여 이쁘게 메세지 전송하기 구현 #10

wants to merge 4 commits into from

Conversation

simnalamburt
Copy link
Member

Closes #4

@simnalamburt simnalamburt self-assigned this Dec 11, 2018
simnalamburt
simnalamburt commented Dec 11, 2018
View reviewed changes
Copy link
Member Author

@simnalamburt simnalamburt left a comment

Choose a reason for hiding this comment

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

아직 다 짠건 아니고 이런식의 방향으로 가려는데 리뷰 부탁드립니다.

그리고 웹훅 호출할때 쓸 HTTP 클라이언트도 아직 못정했는데 그거도 정해주십쑈

@Kroisse
Copy link
Member

Kroisse commented Dec 11, 2018

좋아 보입니다. nickname을 진작에 따로 나눴어야 했었네요.

라이브러리는 특별한 이유 없다면 serenity가 제공하는 걸 그대로 쓰는 편이 낫다고 생각합니다.

@simnalamburt
Copy link
Member Author

@Kroisse 이 프로그램이 설정파일을 어떤 방식으로 읽고 쓰는건지 모르겠어서, 일단 현재와 같은 형태에서 구현을 멈췄습니다. 웹훅 ID와 토큰을 설정에 저장하는 부분만 완성시켜주시고 문서화해주시면 증말 감사드리겠습니다

@Kroisse Kroisse force-pushed the master branch 4 times, most recently from 20feb5e to fcac5e7 Compare September 17, 2019 17:25
@simnalamburt simnalamburt removed their assignment Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kroisse Kroisse Awaiting requested review from Kroisse

Assignees
No one assigned
Labels
enhancement
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

IRC 쪽에서 오는 메시지를 Discord 사용자가 보낸 것처럼 자연스럽게 만들기
2 participants
@simnalamburt @Kroisse