Skip to content

Conversation

@ChaMinXD
Copy link

@ChaMinXD ChaMinXD commented Mar 27, 2024

배포 링크

언어 및 라이브러리


🛠️ Refactoring

UI

  • 글자 수 관련 버그 수정 ( Title Max Length 설정 및 Card Component 수정 )
  • Modal 창 생성시 뒤쪽 Blur 처리
  • Card Components list 반응형 적용

Feature

  • Survey 수정하기 기능 구현
  • useModal Custon Hook 을 통해 Modal 관리
  • LoginModal , SurveyModal Componet 파일 분리
  • form type 에 id 값 ( uuid 라이브러리 활용 ) 추가 -> 추후 기능 추가를 위해

🔑 Structure

Folder File Description
assets arrow svg
search svg
components Card 생성한 설문 정보를 담고있는 Card
Header 페이지 Title & 사용자 닉네임을 보여주는 상단bar
LoginModal 간단 로그인 Modal
SurveyModal 설문 Modal
SurveyTag 설문 modal에서 사용할 태그 컴포넌트
pages HomePage 첫 화면 & 간단 로그인을 실행하는 페이지
MyspacePage 설문을 생성하고 확인하는 페이지
hooks useModal Modal 관리 Custom Hook
router index 라우터 정보를 담고있는 파일

🚀 Future Implementation Plans

  • ⭐Type 전역 관리
  • ⭐태그 생성 및 수정 시 삭제 버튼 추가
  • Protect router 추가 (로그인 하지 않으면 /myspace 이동 차단 및 에러 페이지로 이동)

@@ -0,0 +1,48 @@
import React from 'react';

const colorList = ['bg-[#F04D1D]', 'bg-[#FF7364]', 'bg-[#A75EFF]', 'bg-[#1EBDFE]', 'bg-[#0DCF85]'];
Copy link
Member

Choose a reason for hiding this comment

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

전역 파일로 관리하시기로 했잖아요 ~~

Comment on lines +10 to +24
const handleNextBtn = () => {
if (nickNameError) {
setModalOpen((prev) => !prev);
navigate('/myspace', { state: { nickName: nickName } });
} else {
alert('닉네임이 이상해요');
}
};
const checkNickName = () => {
if (nickName.length < 4) {
setNickNameError(false);
} else {
setNickNameError(true);
}
};
Copy link
Member

Choose a reason for hiding this comment

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

이부분 뭔가 되게 이상하게 읽혀..
닉네임의 길이가 4 이하면 닉네임에러를 false로 세팅하는것도 그렇고
닉네임 에러가 있으면 마이스페이스로 보내는것도 이상해
완전 반대로 해야할거같아

Copy link
Member

Choose a reason for hiding this comment

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

이모지를 입력하면 버그가 생기네
심심할때 해결해봐 !

import SurveyTag from '@components/SurveyTag';
import { v4 as uuidv4 } from 'uuid';

type form = { id: string; title: string; content: string; tagList: string[] };
Copy link
Member

Choose a reason for hiding this comment

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

포매터가 왠지 조금 덜 적용되는거같아

Comment on lines +26 to +28
if (tag.trim() === '') {
return;
}
Copy link
Member

Choose a reason for hiding this comment

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

그냥 리턴해줘 중괄호 빼고 !

Copy link
Member

@kwakseongjae kwakseongjae left a comment

Choose a reason for hiding this comment

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

더 남길거야 기다려

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants