-
Notifications
You must be signed in to change notification settings - Fork 79
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
[장혜민]Sprint9 #644
The head ref may contain hidden characters: "Next.js-\uC7A5\uD61C\uBBFC"
[장혜민]Sprint9 #644
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
과제 하느라 고생하셨습니다. 클래스명 잘 변경 해주셨고, next/router도 잘 활용해주셨네요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.env 적용해주신 점 좋습니다. 다만, .env의 목적은 보안 목적으로 해당 변수들을 깃허브에 올리지 않기 위함이라서 .gitignore에 추가해주시고 올리시지 않아주시면 됩니다.
@@ -7,15 +7,14 @@ function formatDate(value) { | |||
} | |||
|
|||
export default function ArticleList({ articles = [] }) { | |||
console.log(articles.writer); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.log 잘 지워주셨네요!
text-Decoration: none; | ||
padding: 24px 23px; | ||
color: #4b5563; | ||
color: #4B5563; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
color에 컬러 변수를 활용해보시는 것도 추천드립니다! 나중에 유지보수 할때, 색상이 변경되면, 컬러 변수의 값 하나만 변경하면 되기 때문입니다.
const res = await axios.get(`/articles`); | ||
console.log(res); | ||
const router = useRouter(); | ||
const { keyword } = router.query; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
구조 분해 할당으로 query만 잘 꺼내서 사용해주셨네요!
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게