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

[7주차] Hooking 미션 제출합니다. #6

Open
wants to merge 57 commits into
base: master
Choose a base branch
from

Conversation

moong23
Copy link
Member

@moong23 moong23 commented Jun 28, 2023

안녕하세요 Hooking팀의 김문기, 장효신입니다~!

최대한 간단하게 작업하기 위해 노력하였고(간단하지않았지만)
다수의 이미지를 가져다 쓰면서 최소한의 양심을 위해
image sprite 기술을 사용해 나름의 최적화를 하였습니다.

react-hook-form을 통해 회원가입과 로그인 단게에서의 코드 량을 줄였습니다~!

배포링크

hyosin and others added 30 commits May 23, 2023 12:20
[Feature/user] 폼 유효성 로직 추가
[Feature/user] 기본 레이아웃 & api 세팅
@moong23 moong23 changed the title [7주차] Hooking 과제 제출 [7주차] Hooking 미션 제출합니다. Jun 29, 2023
Copy link

@oyatplum oyatplum left a comment

Choose a reason for hiding this comment

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

안녕하세요 Repick 이예지입니다😇

백엔드와 처음 협업하는 과제 하시느라 여러 시행착오가 있으셨을텐데 무사히 마무리 하시느라 수고하셨어요~!
많은 노력이 담긴 코드였던 것 같습니다ㅎㅎ

세션 때 봬요🙏🏻

Comment on lines +24 to +26
message:
"적어도 한개의 대문자, 소문자, 숫자, 특수문자를 포함해서 8자 이상으로 입력해주세요",
};

Choose a reason for hiding this comment

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

오... password에 이런 디테일까지 추가해주셨네용!! 굿

Comment on lines +12 to +23
const [isHovered1, setIsHovered1] = useState(false);
const [isHovered2, setIsHovered2] = useState(false);

const navigate = useNavigate();

const handleMouseEnter = ({ type }: IVRprops) => {
type === 1 ? setIsHovered1(true) : setIsHovered2(true);
};

const handleMouseLeave = ({ type }: IVRprops) => {
type === 1 ? setIsHovered1(false) : setIsHovered2(false);
};

Choose a reason for hiding this comment

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

onMouseLeave를 통해 hoverstate로 처리하는 것도 좋은 아이디어네용!!!

Comment on lines +26 to +31
export const GET = (url: string, params?: any) =>
fetchWrap({ method: "get", url, params });

export const POST = (url: string, body: any, params?: any) =>
fetchWrap({ method: "post", url, body, params });

Choose a reason for hiding this comment

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

오호... 이런 방법이?!

Comment on lines +61 to +65
if (window.confirm(`${person.name}님에게 투표하시겠습니까?`) === true) {
handleVote(person.name);
// handletest(person.name);
} else {
alert("취소되었습니다.");

Choose a reason for hiding this comment

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

투표하기 전 한 번 더 확인하는 방법 좋은 것 같아요!!! ㅎㅎ

Copy link

@sujinRo sujinRo left a comment

Choose a reason for hiding this comment

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

안녕하세요! 댄서포트 팀 노수진입니다.😊

Hooking 팀의 코드가 너무 잘 정리 되어있어서, 리뷰를 하기 좋았고, 또, 이후 있을 협업에서 적용해야겠다고 생각한 부분들이 많이 있었습니다!

이번과제도 수고하셨습니다!👍

Copy link

Choose a reason for hiding this comment

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

공통 color을 따로 정리해두니까 더 편리하고, 협업에 유용할 것 같아요!
팀 플젝때는 이용해야겠어요,,


localStorage.setItem("accessToken", accessToken);
cookies.set("refreshToken", refreshToken, { path: "/" });
};

Copy link

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.

4 participants