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

design : 추천콘텐츠에서 로그인하라는 모달 뜰때 보이는 mockdata 배경에 흰색 더 짙게 넣기 #417

Conversation

godhyzzang
Copy link
Collaborator

@godhyzzang godhyzzang commented Dec 16, 2024

📄 Description of the PR

🔧 What has been changed?

📸 Screenshots / GIFs (if applicable)

<수정 후> (배경에 흰색 더 짙게 추가)
image

⚠️ Precaution & Known issues

✅ Checklist

  • UI 브랜치 같이 확인해서 이슈없는지 확인해보기
  • 함수 이름, 변수 이름만 봐도 어떤 기능을 하는지 파악할 수 있는지 (선언적인 코드인지 확인)

Copy link
Collaborator

@smosco smosco 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 60 to 62
data: { recommendedContents: [] },
data: {
recommendedContents: mockRecommendedData.data.recommendedContents,
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

실패했을 때는 빈배열이 맞는 것 같습니다~ 실패했을 때 목데이터를 불러와 혼란스러울 것 같습니다.
심지어 목데이터가 정확한 데이터가 아니라 이미지와 제목 contentId가 제대로 연결되어 있지 않은 상태입니다.
로그인 했지만 실패했을 때 모달은 안 보여주고 '추천 데이터를 불러오지 못했어요'라는 메세지만 보여주면 좋을 것 같습니다~

@godhyzzang godhyzzang changed the title fix : logout일 때 추천리스트컴포넌트에 mockRecommendedData안 보여주는 문제 해결 design : 추천콘텐츠에서 로그인하라는 모달 뜰때 보이는 mockdata 배경에 흰색 더 짙게 넣기 Dec 16, 2024
@smosco smosco merged commit dcc5f3a into develop Dec 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] resolve the problem that not showing mock recommendedList when not login
2 participants