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

feat: 설정 링크 조회 API v2 구현 #291

Merged

leeeeeyeon merged 2 commits into develop from feature/get-setting-urls-v2

Sep 13, 2024

Collaborator

leeeeeyeon commented Sep 13, 2024 •

edited

Loading

🛰️ Issue Number

🪐 작업 내용

빠른 구현을 위해 Controller와 Service 및 테스트 클래스 이름에 V2를 붙여서 분리하였습니다.
하지만 해당 방법이 최선의 방법이 아니라고 생각이 되어 커스텀 어노테이션 등의 방법으로 추후 리팩토링이 필요합니다.

해당 방법이 최선이 아니라고 생각하는 이유

만약 다른 버전(v1, v2)에서 동일한 함수가 필요할 경우 어느 클래스에 위치할지 애매해지는 문제 발생
버전별 차이가 미미하더라도 새로 코드를 작성해줘야 하므로 중복 코드 발생
버전별 차이를 한눈에 파악할 수 없어 버전별 클래스를 다 확인해봐야 함

📚 Reference

✅ Check List

DB 스키마가 일치하는지 확인했나요?
SonarLint를 반영하여 코드를 수정했나요?
merge할 브랜치의 위치를 확인했나요?
Label을 지정했나요?

leeeeeyeon added 2 commits

September 13, 2024 19:28


          docs: ERD 설정 테이블에 name 컬럼 추가

71e8441


          feat: 설정 링크 조회 API v2 구현 및 테스트 코드 작성

028e1e6

leeeeeyeon added the 🎁 feature label

leeeeeyeon self-assigned this

sonarcloud bot commented Sep 13, 2024

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Collaborator Author

leeeeeyeon commented Sep 13, 2024

Overall Project	41.41%	❌
Files changed	100%	🍏

File	Coverage
Setting.java	100%	🍏
AdminV2Service.java	100%	🍏
AdminV2Controller.java	100%	🍏

leeeeeyeon merged commit e3c8eb7 into develop

2 checks passed

leeeeeyeon deleted the feature/get-setting-urls-v2 branch

September 13, 2024 15:04

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

Labels