[7주차] Team 포토그라운드 강다혜 & 이희원 미션 제출합니다. by hiwon-lee · Pull Request #4 · CEOS-Developers/next-vote-20th

hiwon-lee · 2025-01-04T13:42:23Z

😀😀next-vote-20th : 배포링크😀😀

드디어 20기 마지막 과제가 끝났네요..
하지만 이제부터 포토그라운드 개발이 본격적으로 시작되겠죠?
앞으로의 개발에서 발생하는 크고 작은 문제들이 저희들을 괴롭혀 두피까지 열이오르는 일들 투성이겠지만, 그럼에도 불구하고 프로젝트가 부디 잘 마무리되었으면 정말정말 좋겠어요.

사실 이번 과제도 그런 일들이 계속 일어나서 힘들었답니다. 특히 로컬에서는 잘 되던 에러처리가 배포하면 안되는 문제가 있었습니다.

로컬 ver.	배포 ver.

대체 왜이럴까요? 😀😀 진짜 이해불가.. 계속 무언가 시도해보고 있지만 일단 마감기한이 다와가서 해결하지 못한 채 제출하게 된 것이 안타까울 따름입니다..ㅜ~ㅜ (다혜가 다른방법으로 해결했습니다)

끝으로 세오스에서 한 학기동안 대단한 분들과 생각과 고민을 나누며 공부할 수 있어서 너무나도 영광이었습니다..ㅎㅎ 부족하디 부족한 저는 아직도 배우고 공부할 것들 투성이라 더 공부를 해야할 것 같습니다..ㅎ
암틍간~`😀 CEOS 프론트 20기분들과 운영진분들 다들 정말 수고 많으셨습니다ㅋㅋㅎ 😀😀😀😀😀😀😀😀😀😀😀😀😀새해복많이받으쎄용ㅋㅎ~

everything-flows · 2025-01-04T15:09:28Z

대체 왜이럴까요? 😀😀 진짜 이해불가.. 계속 무언가 시도해보고 있지만 일단 마감기한이 다와가서 해결하지 못한 채 제출하게 된 것이 안타까울 따름입니다..ㅜ~ㅜ (다혜가 다른방법으로 해결했습니다)

왜냐하면... 강다혜가 server action을 또 잘못 이해하고 있었기 때문이죠~ 하하

Programming-Seungwan · 2025-01-05T04:24:34Z

eslint.config.mjs

+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;


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

너무 꼼꼼하게 잘 해주신 것 같아요!!! 최고,, 저는 사실 날래날래(?) 해버려가지구ㅎㅎㅎ 반성하구 갑니다ㅠㅠ!

hae2ni · 2025-01-06T00:53:04Z

src/mocks/http.ts

왤케 꼼꼼해들 당신,,,,,,,,,, 약간 반성하게 되네요,,,, msw라니,,,

hae2ni · 2025-01-06T00:56:11Z

src/app/_components/InputList.tsx

+  font-size: ${({ theme }) => theme.typography.body2_rg.fontSize};
+  font-weight: ${({ theme }) => theme.typography.body2_rg.fontWeight};
+  line-height: ${({ theme }) => theme.typography.body2_rg.lineHeight};


요거 그냥 이렇게 할 필요 없이 한 줄로 가능하지 않나용?

hae2ni · 2025-01-06T00:56:43Z

src/app/_components/InputList.tsx

+  font-size: ${({ theme }) => theme.typography.body2_rg.fontSize};
+  font-weight: ${({ theme }) => theme.typography.body2_rg.fontWeight};
+  line-height: ${({ theme }) => theme.typography.body2_rg.lineHeight};


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};

hae2ni · 2025-01-06T00:58:28Z

src/components/Card/index.tsx

+  size,
+  title,
+  content,
+  voteCount,
+  selected,
+  onClick,


저는 props가 많을 땐 그냥 props를 받은 뒤에 구조분해할당 해주는 편인데 요 방법도 추천드려요!

hae2ni · 2025-01-06T00:59:16Z

src/components/TNB/index.tsx

+import Title from './Title';
+import SubTitle from './SubTitle';
+import Back from './Back';
+


최고닼ㅋㅋㅋ

billy0904

마지막 과제까지 정말 수고 많으셨습니다!
포토그라운드 컨셉에 맞는 UI 색상 사용이 인상적이었네요ㅎㅎ
늘 응원하겠습니다!

(+) 우리 백엔드 한솔이 아니고 한슬인데.....🥹🥹 db에 한솔이라고 들어가있나봐

billy0904 · 2025-01-06T00:04:49Z

src/app/_components/PhoneInput.tsx

+        <PhoneIcon size="1.5rem" />
+        <Input placeholder="전화번호" type="number" />
+      </InputContainer>
+      <Message variant="caption1_rg">{`'-' 없이 입력해주세요`}</Message>


캡션 형식으로 메시지 띄워주는 거 너무 좋은 것 같아요!👍

billy0904 · 2025-01-06T00:08:23Z

src/app/signin/InputList.tsx

+  gap: 1rem;
+
+  padding: 0.5rem 1rem;
+  background: ${({ theme }) => theme.colors.gray[900]};


import 없이 theme을 이렇게 사용하는 방법도 있네요!?😯 요즘 더 나은 theme 사용법을 익히고 있어서...ㅎㅎ 배워가겠습니다!

billy0904 · 2025-01-06T00:12:00Z

src/app/vote/demo-day/screen.tsx

+
+  return (
+    <Container action={handleSubmit}>
+      <Text variant="header1">데모데이 파트장 투표</Text>


데모데이 투표..?

billy0904 · 2025-01-06T03:35:19Z

src/app/vote/front-end/screen.tsx

+  candidateList,
+}: {
+  candidateList: any[];
+}) {
+  const [selectedCandidate, setSelectedCandidate] = useState<Candidate | null>(
+    null,
+  );


위에서는 candidateList 타입을 string[]으로 정의했는데 여기서는 any 로 설정하신 이유가 따로 있을까용?

ryu-won

과제하시느라 수고 많으셨습니다!
재사용 컴포넌트들 정리가 잘되어있었는데, 저는 컴포넌트로 따로 안만들고 한페이지에 다 때려박아서 ... 제 코드를 조금 반성하게 되었습니다!

ryu-won · 2025-01-06T06:53:47Z

src/app/_lib/signin.ts

@@ -0,0 +1,49 @@
+'use server';


'use client'만 있는 줄 알았는데 server도 있군요!!!

ryu-won · 2025-01-06T07:11:51Z

src/app/vote/back-end/action.ts

+    const cookieStore = await cookies();
+    const accessToken = cookieStore.get('access_token')?.value!;
+


저희는 로컬스토리지에 저장해서 가져왔는데 서버에 쿠키로 직접 저장할수도 있군오!

ryu-won · 2025-01-06T07:36:10Z

src/app/vote/back-end/screen.tsx

+    if (result && typeof window !== 'undefined') {
+      window.alert(result);
+    }


해당 페이지는'use client'; 라서 window객체를 확인하는 typeof window !== 'undefined' 코드가 불필요하지 않을까용?

ryu-won · 2025-01-06T07:39:50Z

src/app/vote/demo-day/action.ts

+    const formData = new FormData();
+    formData.set('leaderId', teamName);
+


혹시 formData는 어디에 사용되는 변수일까요?

ryu-won · 2025-01-06T07:47:45Z

src/components/TNB/index.tsx

저였다면 하나하나씩 불러왔을텐데,, 한번에 사용할 컴포넌트들을 이렇게 객체로 묶어주는 방법도 있군요..!

ryu-won · 2025-01-06T07:54:53Z

src/mocks/http.ts

이런식으로 api나오기전에도 Mock api개발이 가능하군요..!

hiwon-lee and others added 30 commits December 26, 2024 15:31

🛠️ build: init project

22a8e6c

Merge branch 'master' of https://github.com/hihi-hye/react-vote-20th

54ad5f3

🛠️ build: 관련 모듈 설치치

8d759cb

✨ feat: 회원가입 요소 구현현

28c7553

✨ feat: 필요한 컴퍼넌트 추가가

70d03ea

📝 docs: assets추가가

b5890b4

✨ feat: route연결결

c362ca4

💄 ui: layout, style추가가

b7004ce

📝 docs: 사용자 model추가가

ad9d384

✨ feat: mocks로 signup 기능 구현현

0abcdd2

💡comment: data넘어온거 확인인

18541c4

🚚 move: 파일위치이동동

1ae96aa

🚚 move: 지운파일복원원

10d81f2

🐛 fix: 파일 이동하면서 에러나는 부분 수정정

5bfe41b

🚚 move: 안쓰는 파일 삭제제

6622ddc

✨ feat: 카드 컴퍼넌트 생성

92d2d72

📝 docs: 데이터 추가가

a6dd5e3

✨ feat: 투표 절차 연결결

75737c3

🚚 move: 드롭다운박스 이름 변경경

16dc1a7

💄 ui: 페이지 높이 수정

322c725

🐛 fix: Input 타입 수정

3afaadc

✨ feat: 회원가입

f3d0f1c

✨ feat: 로그인

2d0d70d

💄 ui: 투표 페이지 UI

22a273c

[fix] build 에러 수정

eaefd02

✨ feat: 투표 기능 (구현중)

8ce8e1d

💄 ui: fe 투표 결과 받아오기

faa7ff4

✨ feat: BE 투표 및 결과 불러오기

58d518d

✨ feat: 데모데이 투표 및 결과 확인

03afe26

✨ feat: 에러처리리

f5dd01e

hiwon-lee changed the title ~~[7주차] Team 페달지니 강다혜 & 이희원 미션 제출합니다.~~ [7주차] Team 포토그라운드강다혜 & 이희원 미션 제출합니다. Jan 4, 2025

hiwon-lee changed the title ~~[7주차] Team 포토그라운드강다혜 & 이희원 미션 제출합니다.~~ [7주차] Team 포토그라운드 강다혜 & 이희원 미션 제출합니다. Jan 4, 2025

🐛 fix: server action 관련 에러 수정

a915c8e

Programming-Seungwan reviewed Jan 5, 2025

View reviewed changes

hae2ni reviewed Jan 6, 2025

View reviewed changes

billy0904 reviewed Jan 6, 2025

View reviewed changes

ryu-won reviewed Jan 6, 2025

View reviewed changes

		const cookieStore = await cookies();
		const accessToken = cookieStore.get('access_token')?.value!;

		const formData = new FormData();
		formData.set('leaderId', teamName);

Conversation

hiwon-lee commented Jan 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

everything-flows commented Jan 4, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hae2ni left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

billy0904 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ryu-won left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

hiwon-lee commented Jan 4, 2025 •

edited

Loading