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

[FIX/#263] 유빈 담당 뷰 / 2차 QA 이슈 대응 #268

Merged
merged 4 commits into from
Mar 22, 2024

Conversation

leeeyubin
Copy link
Member

⛳️ Work Description

  • 취향 태그 라이팅 수정
  • 프로필 조회 글씨 패딩값 수정
  • 엠티뷰 캐릭터 이미지 수정

📸 Screenshot

📢 To Reviewers

  • 2차 QA 수정사항 모두 반영했습니다~!!

@leeeyubin leeeyubin added 유빈 ❄ FIX 🔨 버그 및 오류 해결 labels Mar 21, 2024
@leeeyubin leeeyubin added this to the 2차 스프린트 milestone Mar 21, 2024
@leeeyubin leeeyubin self-assigned this Mar 21, 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 +42 to +46
ProfilePreferenceData("01", "계획은 어느 정도로 세울까요?", "즉흥으로", "철저하게", styleA),
ProfilePreferenceData("02", "장소 선택의 기준은 무엇인가요?", "로컬장소", "관광지", styleB),
ProfilePreferenceData("03", "어느 식당을 갈까요?", "가까운 곳", "유명 맛집", styleC),
ProfilePreferenceData("04", "기억하고 싶은 순간에!", "눈에 담기", "사진 필수", styleD),
ProfilePreferenceData("05", "하루 일정을 어떻게 채우나요?", "여유롭게", "알차게", styleE)
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

@chattymin chattymin Mar 21, 2024

Choose a reason for hiding this comment

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

갑자기 든생각인데 이런 공통되는 데이터는� 로컬 DB를 만들고, 기존에 쓰는 방식처럼 Service, Repository 등등 거쳐서 가져오면 유지보수 짱짱될것 같네요!!

추후 이 텍스트를 서버에서 주게 된다면 경로만 수정해주면 되니까 더 간편해지구용!

Copy link
Member Author

Choose a reason for hiding this comment

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

오홍 넵! 추후에 한번 해보겠습니닷

Copy link
Contributor

@crownjoe crownjoe 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 -122 to +126
ProfilePreferenceData("01", "계획은 어느 정도로 세울까요?", "철저하게", "즉흥으로", styleA),
ProfilePreferenceData("02", "장소 선택의 기준은 무엇인가요?", "관광지", "로컬장소", styleB),
ProfilePreferenceData("03", "어느 식당을 갈까요?", "유명 맛집", "가까운 곳", styleC),
ProfilePreferenceData("04", "기억하고 싶은 순간에!", "사진필수", "눈에 담기", styleD),
ProfilePreferenceData("05", "하루 일정을 어떻게 채우나요?", "알차게", "여유롭게", styleE)
ProfilePreferenceData("01", "계획은 어느 정도로 세울까요?", "즉흥으로", "철저하게", styleA),
ProfilePreferenceData("02", "장소 선택의 기준은 무엇인가요?", "로컬장소", "관광지", styleB),
ProfilePreferenceData("03", "어느 식당을 갈까요?", "가까운 곳", "유명 맛집", styleC),
ProfilePreferenceData("04", "기억하고 싶은 순간에!", "눈에 담기", "사진 필수", styleD),
ProfilePreferenceData("05", "하루 일정을 어떻게 채우나요?", "여유롭게", "알차게", styleE)
Copy link
Contributor

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.

굿굿티비~

@leeeyubin leeeyubin merged commit 26e0d38 into develop Mar 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIX 🔨 버그 및 오류 해결 유빈 ❄
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 유빈 담당 뷰 / 2차 QA 이슈 대응
4 participants