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

부리미 서버 1.10v 배포 #185

Merged
merged 11 commits into from
Feb 19, 2024
Merged

부리미 서버 1.10v 배포 #185

merged 11 commits into from
Feb 19, 2024

Conversation

pp449
Copy link
Member

@pp449 pp449 commented Feb 10, 2024

🤠 개요

  • 건물정보 API 추가
  • 크롤링 병렬처리
  • 무한 슬랙알림 해결

💫 설명

📷 스크린샷 (Optional)

hwinkr and others added 11 commits January 27, 2024 19:03
Feat: 건물 정보 api 라우팅 추가
비동기처리 이외에도 기존에 함수단위의 트랜잭션을 학과 단위의 트랜잭션으로 축소
만약 도중에 탈출하지 않으면 무한 루프로 인해 서버에 안좋은 영향 또는 무한 슬랙 알림을 받기에 3회까지 시도 후 종료되도록 설정
추가적으로 크론 내부에서 트랜잭션을 생성하지 않도록 변경
불필요한 콘솔 로그 제거
얼리 리턴 위치 변경
필터 조건 간결화
fix(majorUtils getDepartmentIdByMajor): 특정 학과의 경우 공지사항 조회 및 알림설정이 불가했던 문제 해결
Fix/#182: 공지사항 크롤링 비동기처리 추가 및 무한 슬랙알림 문제 해결
@pp449 pp449 self-assigned this Feb 10, 2024
@pp449 pp449 requested a review from hwinkr February 12, 2024 07:57
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.

✅ LGTM

@pp449 pp449 merged commit 8ecbaf5 into main Feb 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants