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.

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

[UI/#50] 프로필 뷰 구현 #57

Merged
merged 16 commits into from
Jan 9, 2024
Merged

[UI/#50] 프로필 뷰 구현 #57

merged 16 commits into from
Jan 9, 2024

Conversation

crownjoe
Copy link
Contributor

@crownjoe crownjoe commented Jan 9, 2024

⛳️ Work Description

  • 스크롤 뷰로 프로필 확인할 수 있음

📸 Screenshot

Screen_Recording_20240110_050901_doorip.mp4

📢 To Reviewers

  • 디자인이 덜 나와서 나오는 대로 확인하고 수정하겠습니다.!
  • 서버 나오면 붙이묜 댐

@crownjoe crownjoe added this to the 1차 스프린트 UI 작업 milestone Jan 9, 2024
@crownjoe crownjoe self-assigned this Jan 9, 2024
@crownjoe crownjoe changed the title [UI/#50] 프로필 뷰 구 [UI/#50] 프로필 뷰 구현 Jan 9, 2024
Copy link
Member

@chattymin chattymin left a comment

Choose a reason for hiding this comment

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

너무 잘했어요~~~

Comment on lines 3 to 15
data class ProfileMock(
val profileTitle: String,
val profileSubTitle: String,
val tags: List<String>,
val profileBoxInfo: List<BoxInfo>,
) {
data class BoxInfo(
val title: String,
val first: String,
val second: String,
val third: String,
)
}
Copy link
Member

Choose a reason for hiding this comment

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

내부 변수명만 다른 완벽히 같은 클래스가 이미 존재하네용~
그걸 재활용 하는 것이 좋아보입니다!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니답 재활용하도록 수정했습니다~~~!!!!!!!👍👍

Copy link
Member

@Marchbreeze Marchbreeze left a comment

Choose a reason for hiding this comment

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

LGTM...^^

Comment on lines 14 to 15
private val profileViewModel by viewModels<ProfileViewModel>()
override fun onCreate(savedInstanceState: Bundle?) {
Copy link
Member

Choose a reason for hiding this comment

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

줄바꿈을 권장합니다 ~

Copy link
Member

@Marchbreeze Marchbreeze left a comment

Choose a reason for hiding this comment

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

다시 LGTM ~

Comment on lines 120 to 121


Copy link
Member

Choose a reason for hiding this comment

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

빈줄이 많네요~

@crownjoe crownjoe merged commit 3fc73dd into develop Jan 9, 2024
1 check passed
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.

[UI] 프로필 뷰 / UI 구현
3 participants