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

데코레이터를 통해서 Messneger API 호출하기 #19

Open
osangu opened this issue Apr 18, 2023 · 0 comments
Open

데코레이터를 통해서 Messneger API 호출하기 #19

osangu opened this issue Apr 18, 2023 · 0 comments
Labels
refactor 리팩토링을 진행합니다

Comments

@osangu
Copy link
Member

osangu commented Apr 18, 2023

  1. 데코레이터를 통해서 문자를 보냄
  2. 문자 템플릿은 함수에서 빼내어 Class로 묶어 관리함
  • PING, ISSUE, PULL_REQUEST와 같이 모든 event마다 send_message 함수를 호출하는 것은 반복되는 부분으로써 불필요합니다.
  • 따라서 전송할 message를 return 하도록 하고 handler에서 해당 문자를 보내는 것으로 처리하면 코드가 훨씬 깔끔해질 것 같습니다.
  • 또한 event에 따른 문자열은 따로 Class를 만들어서 정적관리하는 것이 좋을 것 같습니다.
@osangu osangu added the refactor 리팩토링을 진행합니다 label Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링을 진행합니다
Projects
None yet
Development

No branches or pull requests

1 participant