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

코드 응집도를 챙기면서 SSOT로 변경하기 #65

Closed
Tracked by #64
mokhs00 opened this issue Jul 8, 2023 · 0 comments · Fixed by #85
Closed
Tracked by #64

코드 응집도를 챙기면서 SSOT로 변경하기 #65

mokhs00 opened this issue Jul 8, 2023 · 0 comments · Fixed by #85
Assignees
@mokhs00
Labels
P2 priority 2

Comments

@mokhs00
Copy link
Member

mokhs00 commented Jul 8, 2023 •
edited
Loading

year 데이터를 추가하지 않으면 로직상 홈페이지에 노출되지 않는 경우가 있었습니다. 관련해서 People 데이터 타입을 SSOT 바라보도록 리팩토링이 필요해 보입니다.
일부 SSOT를 바라보는 게 이상적이지만 산발적으로 분산된 코드도 있습니다. 해당 코드도 정리가 필요해보입니다. (같은 TAB 데이터를 다루는 코드1, 코드2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mokhs00 mokhs00

Labels
P2 priority 2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor SSOT로 묶을 수 있는 코드 정리 AUSG/2022-homepage
1 participant
@mokhs00