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

Refactor/#81: 중복된 공지사항이 DB에 저장 안되도록 수정 #82

Merged

pp449 merged 3 commits into dev from refactor/#81

Aug 21, 2023

Member

pp449 commented Aug 20, 2023

🤠 개요

closes: Refactor: 테이블을 Unique한 컬럼으로 만들어 중복된 데이터가 저장 안되도록 수정 #81
불필요한 try ~ catch문을 제거했어요
중복된 공지사항이 DB에 저장 안되도록 수정했어요
공지사항 불러올 때 날짜순으로 정렬되어 반환하도록 수정했어요

💫 설명

공지사항 클롤링 시 에러

cheerio에서 문제가 있는지 크롤링하면서 한 번씩 에러가 발생할 때가 있어요
그래서 크롤링 실패하면 크롤링을 중지했었는데, 에러가 발생하면 해당 학과를 제외하고 나머지 학과들은 그대로 크롤링 진행하도록 설정했어요

📷 스크린샷 (Optional)

pp449 added 3 commits

August 20, 2023 23:27


          Feat: 학과, 학교 공지사항 테이블에서 공지사항 URL link에 unique 설정

fed9d57


          Refactor: 필요한 곳에서만 try ~ catch 사용하도록 수정

255096e


          Fix: 공지사항 불러올 때 날짜순으로 정렬하여 불러오도록 수정

b2e010a

pp449 added the 🧑‍🔧 refactor label

pp449 self-assigned this

hwinkr approved these changes

View reviewed changes

Collaborator

hwinkr left a comment

✅ 좋아요~

pp449 merged commit 1fbee84 into dev

1 check passed

pp449 deleted the refactor/#81 branch

August 21, 2023 14:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧑‍🔧 refactor