Skip to content

[FEAT] 데이터 모델 정의 및 데이터 추가 #6

Merged
Peter1119 merged 4 commits intodevelopfrom
feature/data-model-define
Aug 13, 2025
Merged

[FEAT] 데이터 모델 정의 및 데이터 추가 #6
Peter1119 merged 4 commits intodevelopfrom
feature/data-model-define

Conversation

@Peter1119
Copy link
Collaborator

🔗 관련 이슈


✨ 작업 내용

  • 팀원 소개에 필요한 데이터 모델을 정의하고 추가했습니다.

📸 Showcase

변경 전 변경 후
이미지 이미지

📌 이미지가 없다면 이 섹션은 생략해도 됩니다.


📝 참고 사항

Motivation 🥳 (코드를 추가/변경하게 된 이유)

Key Changes 🔥 (주요 구현/변경 사항)

To Reviewers 🙏 (리뷰어에게 전달하고 싶은 말)

  • 대략적인 데이터를 넣어놨기 때문에 머지 후에 자신의 정보를 각자 추가하고 수정하면 될 것 같습니다

Reference 🔗

Close Issues 🔒 (닫을 Issue)

Close #No.

Checklist

  • 브랜치를 가져와 작업한 경우 이전 브랜치에 PR을 보냈는지 확인
  • 빌드를 위해 SceneDelegate 수정한 것 PR로 올리지 않았는지 확인
  • 필요없는 주석, 프린트문 제거했는지 확인
  • 컨벤션 지켰는지 확인
  • final, private 제대로 넣었는지 확인
  • 다양한 디바이스에 레이아웃이 대응되는지 확인
    • iPhone SE
    • iPhone 13
    • iPhone 13 Pro Max

@Peter1119 Peter1119 requested review from Roy-wonji and minneee August 12, 2025 02:23
@Peter1119 Peter1119 self-assigned this Aug 12, 2025
@Peter1119 Peter1119 added the ✨ 기능추가 새로운 기능 추가 label Aug 12, 2025
Copy link
Collaborator

@minneee minneee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!

var introduction: String
var strengths: [String]
var blogLink: String
var collaborationStyle: [String]
Copy link
Collaborator

Choose a reason for hiding this comment

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

피그마에 협업 스타일은 String 형태로 되어있는 상황이라 String으로 변경하는게 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 ~!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

변경했습니다! 확인 부탁드려요 ~!

Copy link
Contributor

@Roy-wonji Roy-wonji left a comment

Choose a reason for hiding this comment

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

@Peter1119 코멘트 단거 확인 하시고 한번 확인 해주세요

}

// MARK: - 초기 데이터 설정
private func setupInitialData(container: ModelContainer) {
Copy link
Contributor

Choose a reason for hiding this comment

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

@Peter1119 이건 뷰모델에서 넣어도 되지 않을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

root view model 생성해서 하자는 말씀이신거죠??
다른 main view model에서 하게 되면
데이터를 넣고 호출하는게 동시에 일어나거나 한 곳에서 일어나서 어색할 것 같아요

Copy link
Contributor

Choose a reason for hiding this comment

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

@Peter1119 흠 그럴수도 있겎네요

# Conflicts:
#	TeamIntroduce/TeamIntroduce/Sources/Presnetaion/IntroduceMain/View/ContentView.swift
@Peter1119 Peter1119 merged commit 31a54a3 into develop Aug 13, 2025
@Peter1119 Peter1119 deleted the feature/data-model-define branch August 13, 2025 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 기능추가 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants