Skip to content

Conversation

@ysw0521
Copy link
Contributor

@ysw0521 ysw0521 commented Apr 13, 2025

No description provided.

Copy link
Member

@iubns iubns left a comment

Choose a reason for hiding this comment

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

상대방의 이름이 나올 수 있도록 해주세요!.
나머진 좋은거 같아요!

Comment on lines +11 to +18
{selectedFriend ? (
<Chat
selectedFriend={selectedFriend}
onBack={() => setSelectedFriend(null)}
/>
) : (
<FriendList onSelectChat={setSelectedFriend} />
)}
Copy link
Member

Choose a reason for hiding this comment

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

이렇게 라우팅 하지 말고 next의 라우팅을 사용해주세요.

Comment on lines +1 to +3
import React, { useState, useRef, useEffect } from "react";
import "./App.css";

Copy link
Member

Choose a reason for hiding this comment

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

해당 프로젝트의 package.json 파일은 어디 갔을까요?

Comment on lines +9 to +13
.App {
display: flex;
flex-direction: column;
height: 100vh;
}
Copy link
Member

Choose a reason for hiding this comment

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

css의 class name 중복을 방지하기 위해
css module을 사용해주세요.

Comment on lines +15 to +29
<Routes>
<Route path="/" element={<Navigate to="/login" />} />
<Route path="/login" element={<Login />} />
<Route path="/register" element={<Register />} />

{/* 채팅 화면 */}
<Route path="/chat" element={
<FriendList onSelectChat={(friend) => setSelectedFriend(friend)} />
} />

{/* 친구 선택 후 이동할 채팅 화면 */}
<Route path="/chat/:friendName" element={
<Chat selectedFriend={selectedFriend} onBack={() => setSelectedFriend(null)} />
} />
</Routes>
Copy link
Member

Choose a reason for hiding this comment

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

router 라이브러리를 따로 사용한거 같네요.
next로 프로젝트를 세팅하면 필요 없습니다.

const navigate = useNavigate();

const handleLogin = () => {
console.log("로그인 시도:", id, pw);
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 +15 to +16
console.log("회원가입 시도:", id, pw);
navigate("/login");
Copy link
Member

Choose a reason for hiding this comment

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

회원가입 API를 연결해주세요.

@iubns
Copy link
Member

iubns commented Jun 2, 2025

이전 과제가 많이 밀려서 한거번에 들어 왔네요.
코멘트를 본다면 여기에 남겨주세요.

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