API 키 관리 기능 추가 및 코드 리팩토링#3
Conversation
H-Genie
requested review from
Kimanna,
hyoungqu23 and
jinsewoon-dev
as code owners
| const { auth } = useAuthStore(); | ||
|
|
||
| const handleCopy = () => { | ||
| const apiKey = document.getElementById('api-key')!.textContent!; |
There was a problem hiding this comment.
어짜피 apiKey를 상태값으로 저장하고 있는 것 아닌가요? 로그인 가능한 계정이 무엇인지 몰라서 테스트는 못해봤지만, 아마 apiKey 상태값의 초기값으로 props가 들어가있고 handleRenew에서 set하고 있어서 apiKey 상태값으로 복사가 가능해보이는데요!
There was a problem hiding this comment.
만약 그게 아니라면 useRef로 가져올 수도 있을 것 같아요!
There was a problem hiding this comment.
직접적으로 태그의 스타일을 변경하는 경우라면 useRef를 사용했겠지만 단순히 값만 읽어오는 것이기에 불필요한 동작이라고 생각합니다
| </figure> | ||
|
|
||
| <nav> | ||
| <nav className='flex flex-col h-[calc(100%-100px)]'> |
There was a problem hiding this comment.
아마 여기도 aside 요소에 flex flex-col 주고 nav 요소에 flex-1 걸어버리면 원하시는 것과 동일한 형태가 나오지 않을까 합니다!
There was a problem hiding this comment.
지난번에 알려주신대로 수정했는데 제가 의도한것과 다르게 동작합니다
알려주신대로 하면 그냥 비율을 유지한채로 폭이 줄어들때 비례해서 줄어드는거고
제가 의도한건 들어들더라도 의도한 최소값은 유지하는거였습니다
이왕 수정한거는 그냥 두겠습니다
There was a problem hiding this comment.
zustand에서 이미 session storage를 persist로 지원하지 않나요?
https://zustand.docs.pmnd.rs/integrations/persisting-store-data
There was a problem hiding this comment.
용법이 다릅니다
zustand는 모든 라우터에서 인증여부를 체크하기 위함이고
sessionStorage는 새로고침 하더라도 인증이 유지되기 위함입니다
H-Genie
force-pushed
the
feature/key-api
branch
from
9223e13 to
fbbb68c
Compare
5 participants
Pull Request
관련 문서
변경 사항
테스트 방법
병합 전 체크리스트