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

Fix/#134: 공지사항 크롤링에 발생하는 문제점 해결 #135

Merged
merged 3 commits into from
Oct 13, 2023
Merged

Fix/#134: 공지사항 크롤링에 발생하는 문제점 해결 #135

merged 3 commits into from
Oct 13, 2023

Conversation

pp449
Copy link
Member

@pp449 pp449 commented Oct 11, 2023

🤠 개요

💫 설명

공간정보시스템공학과 크롤링 에러 처리

  • 공간정보시스템공학과의 경우 1페이지가 모두 고정 공지사항으로 되어있고, 2페이지에 일반 공지사항이 있어서 모든 데이터를 크롤링하지 맛하는 데이터가 있었어요
  • 그래서 해당 학과만 공지사항 2페이지까지 크롤링하도록 추가했어요

고정 공지사항 게시글이 내려가도 DB에 저장되어 있는 문제

  • 일반 공지사항은 게시글이 잘 안내려가지만 고정 공지사항의 경우 변동이 잦을수도 있는데 고정 공지사항이 내려가도 계속 유저에게 계속 보여주는 문제가 있어요
  • 그래서 크롤링했을 때 고정 공지에서 내려갔으면 DB 에서 삭제하도록 로직을 추가했어요

📷 스크린샷 (Optional)

@pp449 pp449 added the 🛠 fix label Oct 11, 2023
@pp449 pp449 self-assigned this Oct 11, 2023
hwinkr
hwinkr approved these changes Oct 13, 2023
View reviewed changes
Copy link
Collaborator

@hwinkr hwinkr left a comment

Choose a reason for hiding this comment

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

✅ 굿굿입니다

@pp449 pp449 merged commit a10f507 into dev Oct 13, 2023
1 check passed
@pp449 pp449 deleted the fix/#134 branch October 13, 2023 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hwinkr hwinkr hwinkr approved these changes

Assignees

@pp449 pp449

Labels
🛠 fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix: 공지사항 크롤링에 발생하는 문제점 해결
2 participants
@pp449 @hwinkr