[7주차] Team 포토그라운드 강다혜 & 이희원 미션 제출합니다. #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hiwon-lee
changed the title
hiwon-lee
changed the title
왜냐하면... 강다혜가 server action을 또 잘못 이해하고 있었기 때문이죠~ 하하 |
Comment on lines
+1
to
+21
| import { dirname } from "path"; | ||
| import { fileURLToPath } from "url"; | ||
| import { FlatCompat } from "@eslint/eslintrc"; | ||
|
|
||
| const __filename = fileURLToPath(import.meta.url); | ||
| const __dirname = dirname(__filename); | ||
|
|
||
| const compat = new FlatCompat({ | ||
| baseDirectory: __dirname, | ||
| }); | ||
|
|
||
| const eslintConfig = [ | ||
| ...compat.extends("next/core-web-vitals", "next/typescript"), | ||
| { | ||
| rules: { | ||
| ...prettier.rules, // Prettier 규칙 추가 | ||
| }, | ||
| }, | ||
| ]; | ||
|
|
||
| export default eslintConfig; |
There was a problem hiding this comment.
Suggested change
| import { dirname } from "path"; | |
| import { fileURLToPath } from "url"; | |
| import { FlatCompat } from "@eslint/eslintrc"; | |
| const __filename = fileURLToPath(import.meta.url); | |
| const __dirname = dirname(__filename); | |
| const compat = new FlatCompat({ | |
| baseDirectory: __dirname, | |
| }); | |
| const eslintConfig = [ | |
| ...compat.extends("next/core-web-vitals", "next/typescript"), | |
| { | |
| rules: { | |
| ...prettier.rules, // Prettier 규칙 추가 | |
| }, | |
| }, | |
| ]; | |
| export default eslintConfig; | |
| import { dirname } from "path"; | |
| import { fileURLToPath } from "url"; | |
| import { FlatCompat } from "@eslint/eslintrc"; | |
| import prettierConfig from "eslint-config-prettier"; | |
| const __filename = fileURLToPath(import.meta.url); | |
| const __dirname = dirname(__filename); | |
| const compat = new FlatCompat({ | |
| baseDirectory: __dirname, | |
| }); | |
| const eslintConfig = [ | |
| ...compat.extends("next/core-web-vitals", "next/typescript"), | |
| { | |
| rules: { | |
| ...prettierConfig.rules, // Prettier 규칙 추가 | |
| }, | |
| }, | |
| ]; | |
| export default eslintConfig; | |
이렇게 코드를 변경해보시면 어떨까요? 그 전에 save dev 옵션으로 eslint-config-prettier 설치해주시는 것 잊지 마시구요!
hae2ni
reviewed
Jan 6, 2025
There was a problem hiding this comment.
왤케 꼼꼼해들 당신,,,,,,,,,, 약간 반성하게 되네요,,,, msw라니,,,
Comment on lines
+29
to
+31
| font-size: ${({ theme }) => theme.typography.body2_rg.fontSize}; | ||
| font-weight: ${({ theme }) => theme.typography.body2_rg.fontWeight}; | ||
| line-height: ${({ theme }) => theme.typography.body2_rg.lineHeight}; |
Comment on lines
+29
to
+31
| font-size: ${({ theme }) => theme.typography.body2_rg.fontSize}; | ||
| font-weight: ${({ theme }) => theme.typography.body2_rg.fontWeight}; | ||
| line-height: ${({ theme }) => theme.typography.body2_rg.lineHeight}; |
There was a problem hiding this comment.
Suggested change
| font-size: ${({ theme }) => theme.typography.body2_rg.fontSize}; | |
| font-weight: ${({ theme }) => theme.typography.body2_rg.fontWeight}; | |
| line-height: ${({ theme }) => theme.typography.body2_rg.lineHeight}; | |
| ${({ theme }) => theme.typography.body2_rg}; | |
Comment on lines
+40
to
+45
| size, | ||
| title, | ||
| content, | ||
| voteCount, | ||
| selected, | ||
| onClick, |
There was a problem hiding this comment.
저는 props가 많을 땐 그냥 props를 받은 뒤에 구조분해할당 해주는 편인데 요 방법도 추천드려요!
| import Title from './Title'; | ||
| import SubTitle from './SubTitle'; | ||
| import Back from './Back'; | ||
|
|
billy0904
reviewed
Jan 6, 2025
| <PhoneIcon size="1.5rem" /> | ||
| <Input placeholder="전화번호" type="number" /> | ||
| </InputContainer> | ||
| <Message variant="caption1_rg">{`'-' 없이 입력해주세요`}</Message> |
| gap: 1rem; | ||
|
|
||
| padding: 0.5rem 1rem; | ||
| background: ${({ theme }) => theme.colors.gray[900]}; |
There was a problem hiding this comment.
import 없이 theme을 이렇게 사용하는 방법도 있네요!?😯 요즘 더 나은 theme 사용법을 익히고 있어서...ㅎㅎ 배워가겠습니다!
|
|
||
| return ( | ||
| <Container action={handleSubmit}> | ||
| <Text variant="header1">데모데이 파트장 투표</Text> |
Comment on lines
+42
to
+48
| candidateList, | ||
| }: { | ||
| candidateList: any[]; | ||
| }) { | ||
| const [selectedCandidate, setSelectedCandidate] = useState<Candidate | null>( | ||
| null, | ||
| ); |
There was a problem hiding this comment.
위에서는 candidateList 타입을 string[]으로 정의했는데 여기서는 any 로 설정하신 이유가 따로 있을까용?
ryu-won
reviewed
Jan 6, 2025
| @@ -0,0 +1,49 @@ | |||
| 'use server'; | |||
Comment on lines
+10
to
+12
| const cookieStore = await cookies(); | ||
| const accessToken = cookieStore.get('access_token')?.value!; | ||
|
|
There was a problem hiding this comment.
저희는 로컬스토리지에 저장해서 가져왔는데 서버에 쿠키로 직접 저장할수도 있군오!
Comment on lines
+52
to
+54
| if (result && typeof window !== 'undefined') { | ||
| window.alert(result); | ||
| } |
There was a problem hiding this comment.
해당 페이지는'use client'; 라서 window객체를 확인하는 typeof window !== 'undefined' 코드가 불필요하지 않을까용?
Comment on lines
+7
to
+9
| const formData = new FormData(); | ||
| formData.set('leaderId', teamName); | ||
|
|
There was a problem hiding this comment.
저였다면 하나하나씩 불러왔을텐데,, 한번에 사용할 컴포넌트들을 이렇게 객체로 묶어주는 방법도 있군요..!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
😀😀next-vote-20th : 배포링크😀😀
드디어 20기 마지막 과제가 끝났네요..
하지만 이제부터 포토그라운드 개발이 본격적으로 시작되겠죠?
앞으로의 개발에서 발생하는 크고 작은 문제들이 저희들을 괴롭혀 두피까지 열이오르는 일들 투성이겠지만, 그럼에도 불구하고 프로젝트가 부디 잘 마무리되었으면 정말정말 좋겠어요.
사실 이번 과제도 그런 일들이 계속 일어나서 힘들었답니다. 특히 로컬에서는 잘 되던 에러처리가 배포하면 안되는 문제가 있었습니다.
대체 왜이럴까요? 😀😀 진짜 이해불가.. 계속 무언가 시도해보고 있지만 일단 마감기한이 다와가서 해결하지 못한 채 제출하게 된 것이 안타까울 따름입니다..ㅜ~ㅜ (다혜가 다른방법으로 해결했습니다)
끝으로 세오스에서 한 학기동안 대단한 분들과 생각과 고민을 나누며 공부할 수 있어서 너무나도 영광이었습니다..ㅎㅎ 부족하디 부족한 저는 아직도 배우고 공부할 것들 투성이라 더 공부를 해야할 것 같습니다..ㅎ
암틍간~`😀 CEOS 프론트 20기분들과 운영진분들 다들 정말 수고 많으셨습니다ㅋㅋㅎ 😀😀😀😀😀😀😀😀😀😀😀😀😀새해복많이받으쎄용ㅋㅎ~