Closed LimEunSang closed 2 years ago
제가 언급한 사항들은 권고사항이기 때문에 바꿀 필요까지는 없어보여 LGTM! 단 console.log는 삭제가 필요하지만 추후에 더 작업하면서 쓰일거 같아 일단 merge해도 괜찮을거 같습니다. 추가로 admin 전체적인 페이지에 로그인을 확인하는 코드가 들어가게 되는데( 작업하신 페이지 포함 제가 작업한 페이지들까지 전부 ) 통일해야 할 필요가 있어보입니다. 일단은 각 페이지별로 useEffect에서 확인하는데 어떻게 합칠 방법이 없을까요??
상당히 많은 양인데 고생하셨습니다~ 3따봉 드립니다.
로그인 확인하는 코드같은 경우 저도 통일화의 필요성을 느껴서 생각만 간단하게 해봤는데 함수화해서 상위 컴포넌트에서 하위 컴포넌트로 전달하는 방식이 어떨까 생각합니다. 백앤드 연동 시 혹은 전체 리팩토리 시 적용해봅시다!
LGTM! 작업 양이 많았을거같은데 수고하셨습니다. 저도 상대경로를 사용했는데 참고하겠습니다 그런데 원래 이 페이지 담당이 은상님이었던가요...?
LGTM! 작업 양이 많았을거같은데 수고하셨습니다. 저도 상대경로를 사용했는데 참고하겠습니다 그런데 원래 이 페이지 담당이 은상님이었던가요...?
아뇨 다른분 담당인데 제가 했어요!