[Feat/user] 공통컴포넌트 추가 및 친구추가 기능 #299
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
minisundev
reviewed
Nov 20, 2024
| @@ -10,6 +10,7 @@ axios 인터셉터로 위의 로직대로 코드 작성 하다가 요청 인터 | |||
| token.ts의 두 함수(validateAccessToken, refreshAccessToken)에서 요청을 보내는 부분이 인터셉터에 의해 다시 가로채지고 있는 것으로 추정 | |||
| 이를 해결하기 위한 방법으로 아래의 인스턴스를 별도 만들어서 해결함 | |||
There was a problem hiding this comment.
맵과 아바타 뿐만이 아니고 Bee 도 만드셨군요 대단해요!
Comment on lines
+10
to
+11
| userId: string | null, | ||
| token: string | null |
There was a problem hiding this comment.
그냥 string으로만 지정하면 안 되고 null일 가능성도 배제하지 않는 이유가 있나요?
| userId: string | null, | ||
| token: string | null | ||
| ) => { | ||
| const url = `${process.env.REACT_APP_BASE_URL}/user/api/v1/user/${userId}/requests`; |
There was a problem hiding this comment.
user/api/v1이 파일 전체에서 반복되는 것 같은데 상수로 지정해두고 사용하면 v2가 나온다거나 코드 리팩토링할때도 편리하지 않을까요?
| placeholder="친구검색" | ||
| value="" | ||
| type="text" | ||
| onChange={onChageSearchFriends} |
There was a problem hiding this comment.
여기에서 onChange 메서드를 사용해서 검색할때마다 value를 업데이트 하려고 하신건가요? 그런데 위에 onChangeSearchFriends 메서드는 비어있는 것으로 보이는데 그 이유는 뭔가요??
Comment on lines
+9
to
+26
| const [newFriends, setNewFriends] = useState< | ||
| GetRequestedFriendsResponseType | undefined | ||
| >(undefined); | ||
|
|
||
| const getNewRequetedFriends = async () => { | ||
| try { | ||
| const response = await getRequestedFriendsData(userId, token); | ||
| if (response) { | ||
| setNewFriends(response); | ||
| } | ||
| } catch (error) { | ||
| console.error(error); | ||
| } | ||
| }; | ||
|
|
||
| useEffect(() => { | ||
| if (token && userId) getNewRequetedFriends(); | ||
| }, [userId, token]); |
There was a problem hiding this comment.
제가 리액트를 잘 몰라서 그러는데 newFriends에 useState를 쓴 이유가 무엇인가요? useEffect나 useRef 말고요! 저는 userEffect와 useRef로 모든 것을 해결해와서 useState의 장점이나 쓰면 좋을 상황이 궁금합니다!@
| @@ -105,7 +107,7 @@ const CreateServerForm = () => { | |||
| <ServerThemeSelector /> | |||
| </FormControl> | |||
|
|
|||
| <Button type="submit">서버생성</Button> | |||
| <button color="bg-sky-600">서버생성</button> | |||
| </Box> | |||
| </form> | |||
| </div> | |||
There was a problem hiding this comment.
신규 생성 모달창이라면 컴포넌트로 분리해두고 ReactElement나 ReactNode로 변경되는 부분만 이 페이지에서 주입해줄 수는 없을까요?!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#294
📝작업 내용
스크린샷 (선택)
💬리뷰 요구사항(선택)
현재 프로젝트에 스타일 파일도 여러개있고 폴더랑 파일들이 정리가 안되어 있어서 정리하면서 만들고 있습니다. MUI 러닝커브도 있고 딱히 편의성을 못 느껴서 MUI라이브러리 제거하고 커스텀 컴포넌트로 관리할까 합니다.