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

Refact: 코드 리팩토링 하기 #9

Open
1 of 3 tasks
todoni opened this issue Dec 29, 2023 · 0 comments
Open
1 of 3 tasks

Refact: 코드 리팩토링 하기 #9

todoni opened this issue Dec 29, 2023 · 0 comments

Comments

@todoni
Copy link
Owner

todoni commented Dec 29, 2023

Idea

  1. http 요청 하는 부분 모듈로 분리할 수 있을 것 같다
  2. 이벤트 종류가 많아지더라도 확장성 있게 코드 짤 수 있게 재구조화 해야함 (ex. dict로 관리한다든지.. if elif는 좀 ㅜ )
  3. 좀 더 범용적으로... 지금은 딱 버튼클릭 했을 때! 저 모달! 만 쓸 수 있는거니까.. 잘 생각해서 짜보자

Todo

  • channel 이름 하드코딩 하지말고 변수화
  • http 요청 모듈 분리
  • lambda_handler에는 딱 lambda_handler만 있게 하고 그 이외 함수들은 좀 더 범용성 있게 추상화 시켜서 모듈 import 하는식으로 쓰자

흠 이러면 내가 만든 slack client와 lambda용 client가 또 생기는 느낌인데. . 어떻게 할지 고민 해봐야 할듯

@todoni todoni added this to scrum-bot Dec 29, 2023
@todoni todoni moved this to Todo in scrum-bot Dec 29, 2023
@todoni todoni moved this from Todo to In Progress in scrum-bot Dec 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

No branches or pull requests

1 participant