Skip to content

빈자리알림 신규 수강신청 제도 적용#485

Merged
asp345 merged 5 commits intodevelopfrom
feature/new-vacancy-noti
Jan 24, 2026
Merged

빈자리알림 신규 수강신청 제도 적용#485
asp345 merged 5 commits intodevelopfrom
feature/new-vacancy-noti

Conversation

@asp345
Copy link
Member

@asp345 asp345 commented Jan 20, 2026

수강스누 공지사항에 있는 강의매매 방지를 위한 수강신청 제도 개선 안내 카드뉴스에 따라서 취소여석 강좌의 수강신청이 특정 시간대에만 가능하게 되었는데 (그 시간대에는 취소여석 강좌 버리기가 안됨)
이에 따라서 빈자리알림 보낼때 어느 시간에 수강신청을 해야 하는지를 알려주고, 여기에 더해 어드민으로 학기별 수강신청 기간을 지정해서 빈자리알림이 자동으로 원하는 시기에만 돌도록 수정했습니다
예시:
"OOOO" 강의에 빈자리가 생겼습니다. 15:00에 수강신청 사이트를 확인해보세요!
오늘 중으로 수강신청 가능한 시간대가 없다면
"OOOO" 강의에 빈자리가 생겼습니다. 다음 수강신청 일자에 수강신청 사이트를 확인해보세요!
라는 식으로 문구 변경

@asp345 asp345 requested review from a team and PFCJeong as code owners January 20, 2026 06:23
@asp345 asp345 requested review from Hank-Choi, davin111 and eastshine2741 and removed request for a team January 20, 2026 06:23
Copy link
Member

@Hank-Choi Hank-Choi left a comment

Choose a reason for hiding this comment

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

솔직히 어드민에서 직접 수강신청기간 등록하는거 반대인데 일단은 없는것보단 나으니까 반영해보죠

}
}
}
delay(DELAY_PER_CHUNK)
Copy link
Member

Choose a reason for hiding this comment

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

딜레이는 왜 다시 생긴거야?

Copy link
Member Author

@asp345 asp345 Jan 24, 2026

Choose a reason for hiding this comment

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

이제 빈자리알림 띄우자마자 수강신청하러 들어가는게 아니라서 좀더 여유롭게 해도 될거같았어
드랍 시간대 끝나고 10분동안 여유도 있고

Comment on lines +52 to +53
.getByYearAndSemester(latestCoursebook.year, latestCoursebook.semester)
?.registrationPeriods ?: throw RegistrationPeriodNotSetException
Copy link
Member

Choose a reason for hiding this comment

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

최대한 기본값 나둬야할 것 같은데
한번도 snutt가 수동으로 관리가 된 적이 없긴해 멤버도 계속 바뀌고

@asp345
Copy link
Member Author

asp345 commented Jan 24, 2026

@Hank-Choi 수동으로 관리하는거 안좋은거 같아서 배치 돌때 수강스누 메인화면에서 기간 파싱하는거 추가했어
예전에 파싱해뒀던건 안사라지게 하는식으로

Copy link
Member

@Hank-Choi Hank-Choi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@asp345 asp345 merged commit 675bb39 into develop Jan 24, 2026
2 checks passed
@asp345 asp345 deleted the feature/new-vacancy-noti branch January 24, 2026 04:32
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