SMH-88 로그인 전 후 레이아웃 #31
Conversation
ysnarmi
requested review from
samilHero,
juchanhwang,
joshhhso and
qkdflrgs
as code owners
There was a problem hiding this comment.
정적인 파일들, 즉 로고, 파비콘 같은 경우는 src 폴더 바깥 public폴더에 저장하는게 어떨까요!
추가로, src 내부에서 사용할 이미지 파일들은 src > assets하위에서 관리하는 것이 어떨까요!
src > assets > icons
그리고, styles 폴더 하위의 파일은 css파일 위주로 사용하는 것이 어떨까요!
There was a problem hiding this comment.
서버사이드와 클라이언트 사이드를 구분하기 위한 컴포넌트 분리인 것으로 이해했습니다!
페이지에 속한, 즉 독립적인 컴포넌트가 아니라면 src > components폴더에서 제외하는 것이 어떨까요?
공용으로 사용하는 컴포넌트만 여기서 관리하고, 외부에서 관심이 없는 컴포넌트는 사용하는 곳에 고립 시키는 것이 관리 포인트에서 좋다고 생각합니다!
app > login > _components > login.tsx 이런 방식으로요!
| return ( | ||
| <> | ||
| <Login /> | ||
| </> |
There was a problem hiding this comment.
Fragment로 감쌀 필요가 있을까요?
const LoginPage = () => <Login />이렇게 써도 될 것 같아요!
|
반영이 완료된 것은 리졸브해주시면 좋을 것 같습니다! |
| if (isErrorLogin) { | ||
| alert(dataLogin); | ||
| } | ||
| }, [dataLogin]); |
There was a problem hiding this comment.
의존성 배열에 isSuccessLogin와 isErrorLogin는 없어도 괜찮은 의존성인가요?
| const { setLogin } = useAdminAPI(); | ||
|
|
||
| const mutationFn = async (request: any) => { | ||
| const { data: response } = await setLogin(request); |
There was a problem hiding this comment.
혹시 api 에러가 발생하면 어떻게 동작할까요?
Jira
배경
주요 기능
사용 예시
기능에 대한 사용 예시를 자유롭게(텍스트, 코드, 스크린샷) 작성합니다.
기타