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

Service layer에서 발생한 셀레니움 관련 예외가 controller까지 던져지지 않도록 수정 #15

Merged

Wo-ogie merged 3 commits into develop from feature/14-exception-catch

Jul 17, 2023

Member

Wo-ogie commented Jul 17, 2023

🔥 Related Issue

Close Service layer에서 발생한 Selenium 관련 예외가 controller layer까지 던져지지 않도록 코드 수정 #14

🏃‍ Task

Service layer에서 발생한 셀레니움 관련 예외가 controller까지 던져지지 않도록 수정

📄 Reference

None

✅ Check List

PR의 제목은 팀 내 규칙을 준수하여 알맞게 작성하였는가?
Merge 하는 브랜치가 올바른가?
팀의 코딩 컨벤션을 준수하는가?
PR과 관련없는 변경사항이 들어가지는 않았는가?
내 코드에 대한 자기 검토가 되었는가?
Reviewers, Assignees, Lables, Project, Milestone은 적절하게 선택하였는가?
관련한 issue를 닫아야 하는지 점검해보고 적용했는가?

Wo-ogie added 3 commits

July 18, 2023 06:31


          refactor: #14 Service layer에서 발생한 셀레니움 관련 예외가 controller까지 던져지지 않도록 수정

4ad64e7


          test: #12 disabled 처리되어 불필요한 테스트 코드 삭제

2832f9f


          test: #14 장소 정보 scraping 기능 구현 코드 변경사항(4ad64e7) 테스트 코드 적용

b214741

Wo-ogie added the ♻️ refactor label

Wo-ogie self-assigned this

Wo-ogie merged commit 9c7860c into develop

1 check passed

Wo-ogie deleted the feature/14-exception-catch branch

July 17, 2023 21:35

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

Labels

♻️ refactor