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작업 #107

Merged
merged 13 commits into from
Jul 21, 2024
Merged

마을 기사 읽기 페이지 UI작업 #107

merged 13 commits into from
Jul 21, 2024

Conversation

stopmin
Copy link
Contributor

@stopmin stopmin commented Jul 20, 2024

관련 이슈

작업 요약

  • 마을 기사 읽기 페이지를 구성합니다.

작업 상세 설명

  • 기존에 기사 읽기 페이지와 크게 다르지 않습니다.
  • 헤드라인/밑에 기사 추가

리뷰 요구 사항

  • 코드 스타일

미리 보기

Screenshot 2024-07-21 at 6 36 19 AM Screenshot 2024-07-21 at 6 36 28 AM

@stopmin stopmin requested a review from yeonddori July 20, 2024 21:36
@stopmin stopmin self-assigned this Jul 20, 2024
Copy link

vercel bot commented Jul 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gyeongdan-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2024 5:59pm

viewCount: number;
source: string;
uploadDate?: string;
viewCount?: number;
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 uploadDateviewCount가 마을에서는 필수가 아닌가요?!
필수라고 생각했어서 여쭤봅니다~!~!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

흠흠쓰 필수로 하겠습니다!! 어제 새벽에 짜서 ㅋㅋㅋㅋㅋ 약간 좀잡을 수 없네요

Comment on lines 50 to 52
<Box sx={{ marginTop: -2 }}>
<KakaoMap villages={[village]} />
</Box>
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 지도 크기도 기사 너비에 맞게 줄이는 것에 대해 어떻게 생각하시나요~~?
너무 혼자 난 지도야!!! 소리치는 느낌이네요😂

대충 이런..? 근데 너무 휑한 것 같기도 하네요 ㅋㅋㅋ 디자인 너무 어렵습니다..
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

흠흠쓰... 일단 한번 수정해보고 판단하개보겠습니다!!!!

import hiddenGems from '@/mocks/villages';

const Page = async ({ params }: { params: { id: number } }) => {
const village = hiddenGems[params.id];
Copy link
Contributor

Choose a reason for hiding this comment

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

데이터가 id는 1부터 시작인데 배열 형태로 쓰셔서 인덱스가 이상하게 잡히네요!
hiddenGems[params.id - 1] 해주셔야 될 것 같아요~!

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
Contributor

@yeonddori yeonddori left a comment

Choose a reason for hiding this comment

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

giphy

저것만 고쳐주시면 될 것 같아요~!~! 잘 짜시네용👍

import color from '@/constants/color';
import hiddenGems from '@/mocks/villages';

const Page = async ({ params }: { params: { id: number } }) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Client components cannot be async functions. 라고 합니다!!
async 빼주세요~!

@stopmin stopmin merged commit 5d8bbe1 into main Jul 21, 2024
3 checks passed
@stopmin stopmin deleted the feature/ISSUE-104 branch July 21, 2024 18:00
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작업
2 participants