Skip to content

[Style] #195 - 마이페이지 완료#199

Merged
OneTen19 merged 32 commits into
developfrom
style/#195
Jan 29, 2026
Merged

[Style] #195 - 마이페이지 완료#199
OneTen19 merged 32 commits into
developfrom
style/#195

Conversation

@OneTen19
Copy link
Copy Markdown
Member

🌴 작업한 브랜치

style/#195

✅ 작업한 내용

  • 마이페이지 현재 진행 못 하는 몇 개 빼고 끝냈습니다!
  • 자격증 수정 및 삭제는 등록 API가 연결 돼야 연결할 수 있어서 후순위로 미뤘습니다. 등록 다 붙이면 하겠슴
  • 로그아웃은 토큰 재발급 로직 구현해야 할 수 있어서 별도 이슈 파서 애플로그인 붙일 때 같이 진행하겠습니다
  • 전에 나연이가 이미지 업로드 하겠다고 해서 해당 부분은 구현하지 않았습니다

❗️PR Point

  • 이슈 단위 좀 나눠서 할까 고민하다가 시간 없어서 그냥 통으로 올렸어요
  • 2500줄 이상의 대형 pr이라 리뷰하기 매우 힘들텐데 귀찮으면 걍 날 믿어.

📟 관련 이슈

@OneTen19 OneTen19 added this to the [CERTI] 1차 스프린트 milestone Jan 27, 2026
@OneTen19 OneTen19 requested a review from sangyup12 January 27, 2026 08:45
@OneTen19 OneTen19 self-assigned this Jan 27, 2026
@OneTen19 OneTen19 added Add ✚ 코드, 파일, 에셋 추가 Feat 💻 기능 구현 labels Jan 27, 2026
@OneTen19 OneTen19 linked an issue Jan 27, 2026 that may be closed by this pull request
5 tasks
@OneTen19 OneTen19 added Network 📡 서버 통신 Refactor 🏗️ 코드 리팩토링 한열 🧄 한열 작업 labels Jan 27, 2026
@OneTen19 OneTen19 requested a review from Yeonnies January 27, 2026 08:45
Copy link
Copy Markdown
Contributor

@sangyup12 sangyup12 left a comment

Choose a reason for hiding this comment

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

와 API 짱많아요... useCase 지옥이다. 고생하셨습니다.! 왕피알 열심히 읽었습니닷

private init() { }


/// TODO: - 매번 API 연결할 때마다 service랑 repository에 case 만들고 usecase 만들고 DIContainer에서 생성해서 일일이 의존성 주입해주려니 너무 귀찮음;;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P3
인정. 도메인별로 UseCase를 하나로 묶어서 한 번에 주입하는 식으로 하는 건 어떨까요. 그럼 도메인마다 한 번씩만 주입해주면 되고 UseCase 추가/삭제할 때도 편할 거 같아요.
근데 한 도메인의 모든 UseCase를 하나의 파일에서 관리해도 괜찮을까요? 어떻게 생각하시나여

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
저도 너무 힘들어서 이번에 앱잼때 멘토님께 여쭤봤는데 너무 애매한 답변만 해주더라고여..

찾아보니까 제네릭한 레포지토리를 만들어서 관리해주는 방법이 있더라고요! 레포지토리 패턴 검색하니까 좀 나오는데
https://damagucci-juice.tistory.com/entry/iOS-CoreDataRepository-Pattern-with-CoreData
아직 이해는 잘 안돼여

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

우리가 레포지토리의 역할에 대해 완전히 잘못 알고 있었더라구요
지금은 그냥 네트워크 연결용 브릿지 느낌으로 쓰고 있는데 실은 좀 더 복합적인 역할을 수행하고 유즈케이스도 마찬가지로 지금처럼이 아니라 다른 방식으로 쓰는 게 좀 더 클아의 철학에 가까운데 이미 너무 프로젝트가 커져버려서 엄두 안남

별도로, 생각해보니 스프린트 하면서 사라진 카테고리랑 추천 쪽 API가 레거시 코드로 남아있어서 좀 더 심하게 느껴지는 듯.
얘네 다 지워버리면 좀 깔끔하긴 할 텐데 뭐가 쓰는거고 뭐가 안 쓰는건지 모르겠어서 그냥 냅뒀어요

Comment on lines +11 to +16
func toServerFormatString() -> String {
let formatter = DateFormatter()
formatter.dateFormat = "yyyy.MM.dd"
formatter.locale = Locale(identifier: "ko_KR")
return formatter.string(from: self)
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

앗 이거 저도 만들었는데 머지 받으면 삭제하고 이거 사용할게요

Copy link
Copy Markdown
Contributor

@Yeonnies Yeonnies left a comment

Choose a reason for hiding this comment

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

잘봤습니다!!
너무 길어서 힘들었는데 믿겠습니다..

ps. 내가 프리사인드 url 하겠다고 언제했는지 기억도 안나는데 빨리 해보겠슴닷

private init() { }


/// TODO: - 매번 API 연결할 때마다 service랑 repository에 case 만들고 usecase 만들고 DIContainer에서 생성해서 일일이 의존성 주입해주려니 너무 귀찮음;;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
저도 너무 힘들어서 이번에 앱잼때 멘토님께 여쭤봤는데 너무 애매한 답변만 해주더라고여..

찾아보니까 제네릭한 레포지토리를 만들어서 관리해주는 방법이 있더라고요! 레포지토리 패턴 검색하니까 좀 나오는데
https://damagucci-juice.tistory.com/entry/iOS-CoreDataRepository-Pattern-with-CoreData
아직 이해는 잘 안돼여

}
.padding(.leading, 20)

DatePickerBox(selectedDate: $selectedDate)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
피그마 보니까 이거 캘린더 커스텀이네유...... 원래 그랬나

print("========== [수정 내용 확인] ==========")
print("📝 자격증 이름: \(item.certificationName)")

if let date = selectedDate {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
저는 이 과정까지 익스텐션 안에 넣었는데 따로 빼야하나요?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ㄴㄴ 굳


final class DefaultMyPageFactory: MyPageFactory {

private let fetchMyPageInfoUseCase: FetchMyPageInfoUseCase
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
뜨헉 난리났다..


import SwiftUI

struct JobCategoryFilterModal: View {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p2
옆에 몇지망인지 1, 2, 3 숫자 동글뱅이 붙는 거 구현 안된거 같은데 맞나요?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

아뇨 구현됐습니다! 작업방 정독 ㄱ

showDeleteAlert = false
}
)
.zIndex(2)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

p3
우와 이건 뭔가요?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

zstack을 사용하면 아래쪽에 작성된 뷰가 위쪽 계층으로 올라온다는 룰이 있긴 하지만
zIndex를 통해 순서를 임의로 정해줄 수도 있습니다!

@OneTen19 OneTen19 merged commit 0c87486 into develop Jan 29, 2026
@OneTen19 OneTen19 deleted the style/#195 branch January 29, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Add ✚ 코드, 파일, 에셋 추가 Feat 💻 기능 구현 Network 📡 서버 통신 Refactor 🏗️ 코드 리팩토링 한열 🧄 한열 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SCRUM-42] Style: 마이페이지 API 연결

3 participants