Fix: 220동 식당 Identifier 추가 & 슬랙 노티 버그

[GoGiants1]

Pull Request

설명

• 슬랙 노티가 4개월 넘게 도달하지 못하고 있었습니다. 크게 2가지 이유였고, 이 PR에서 모두 해결했습니다.

  1. 기존의 봇이 비활성화된 상태 -> 새로운 봇 생성 후 교체
  2. slack 메세지 보내는 함수에서 channel 값이 토큰값으로 잘못 보내지고 있었음

• 220동 식당의 경우 전화번호가 등록되어 있지 않아서, 오류가 일부 발생하고 있었습니다. 전화번호를 추가하여 해결하였습니다.

• 수의대식당의 경우 자체 홈페이지 연결이 불안정하여 ssl 인증 오류가 발생하는 상태가 반복(설정에는 ssl 인증을 False로 해두었지만..)되었습니다. 그래서 base_crawler.py 의 run 함수에서 에러 핸들링 로직을 밖으로 빼내고, 연결 오류도 에러핸들링과 로깅(어떤 URL에서의 오류인지 확인) 할 수 있게 했습니다.

• snuco의 경우 baseurl이 변경되어, 최신화했습니다.

해결한 이슈가 있다면 아래에 이슈 번호들을 적어주세요.
Closes #85

어떤 종류의 PR인가요?

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

테스트는 어떤 방식으로 진행하셨나요?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Unit Test with pytest
• Integration Test with pytest
• Manual Test

체크리스트

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works

@invalidid56 @tictactoeid @GanziDaeyong @soobin-jeon 리뷰 부탁드립니다.

[GoGiants1]

버그 수정을 위해 먼저 머지하겠습니다.

[GanziDaeyong]

해외여행중이라 늦게 봤습니다 ㅠ 헉 이거 내역 보니 슬랙 노티와 220동 제 실수였네요.. 트롤링 죄송합니다. 🧇 @GoGiants1