전남대 FE_박건규 6주차 과제 Step1,2 #99
Open
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
baseURL의 데이터는 해당 데이터를 메인으로 사용하는 axios instance를 만드는 모듈에 둠 baseURL의 리스트를 객체로서 관리하도록 하였으며, 이는 사용시 BASE_URL[serverType] 로 사용할 수 있다. 타입 지정시 keyof typeof BASE_URL로 할 수 있다. baseURL관리는 전부 useServer hook으로 위임하였으며, 이는 상태 및 session storage에서 동시에 저장하도록 하였다. hook을 사용할 수 있는 곳에서는 state로서 반환받는 값을 사용하여 빠르게 가져올 수 있으며, 사용하지 못하는 서버단에서는 sessoinStorage에서 가져오도록 설계하였다.
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.
바쁘신 와중에도 코드리뷰를 해주시는 멘토님께 감사의 말씀 드립니다.
배포 링크 : https://geongyu09-step3.react-deploy-d5u.pages.dev/
(3단계까지 진행한 코드가 배포되어 있습니다.)
해결과정
1단계 :
2단계 :
배포
리뷰어에게
1단계는 코드가 조금은 더러운 느낌이 드네요.. 그래서 리뷰를 부탁드리기가 죄송스럽네요.
아무래도 빠르게 백엔드와 기능을 맞추어 봐야해서, 바로 3단계로 넘어가게 되었고, 여기서 1단계 브랜치를 변경하면 컨플릭 해결에서 조금 비용이 들어갈 것 같아서 손을 떼버리니 이렇게 된 것 같습니다. 수정사항이 있다면 3단계 pr에서 수정하도록 하겠습니다