-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release 0.2 #96
Merged
Release 0.2 #96
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
api 요청은 모두 /api 경로를 거치도록 리버스 프록시가 설정되었기 때문에 라우팅 주소가 변경되었음
api 주소를 constants.ts 로 넣어놨고 배포용 api 서버 주소는 보안을 위해 .env 파일에서 관리함
thumbnailUrl, originUrl을 optional 로 수정함.
이미지 데이터의 creator가 특정 유저의 oid 값을 갖도록 수정함
메인에 보여줄 카드뷰에 creator를 표시해주기위해 populate 기능 추가함
object storage의 원본이미지가 아닌 image optimizer로 압축된 이미지 Url을 가져오도록 설정
서버에서 보내는 response를 일관된 형식으로 사용할 수 있도록 utils/response.ts 작성 response(res, {data}) 형태로 사용 가능 에러가 난 경우는 exporess error handler에서 처리함 에러 메시지는 utils/messages.ts 에 넣어두고 import 해서 사용 error handler에서 response 함수를 리턴하면 마지막에는 status code를 200으로 바꾸어 응답을 내려준다. 대신 브라우저에서는 status code 200을 받기 때문에 try-catch로 처리가 불가능 함.
American palette - https://flatuicolors.com/palette/us
전역에서 사용가능하도록 box-shadow 갑 세팅
useFetch가 다루는 데이터에 대한 의존성을 없애기 위해 Generic Type 사용하도록 변경
삼항연산자 사용으로 코드 가독성 높임
구조분해 할당사용
/src/upload --> /src/UploadMain 으로 변경 /UploadMain/index.tsx /UploadMain/style.tsx 식으로 모든 업로드 관련 컴포넌트를 위와같은 형식으로 분리함
스타일 컴포넌트를 사용하면 className으로 판별해서 스타일을 먹일필요 없음
Feature/jmj/review feedback
…feature/ljh/main-page
api 주소관리 관련
style 파일을 분리함
theme에 칼라 추가 및 theme 파일 활용
useState 활용 ( https://velog.io/@velopert/use-typescript-and-redux-like-a-pro) 참고 리덕스의 상태값 state.login의 userOid 값을 확인해서 있으면 홈으로 리다이렉트,, 로그인 페이지 자체에서 이 값을 확인해서 리다이렉트하도록함 로그인되어있을 경우 로그인 페이지로 못들어오게 됨
jwt 토큰을 검증하고 디코드 하는 미들웨어를 만들고 다른 api들이 이 미들웨어를 활용하도록 구조를 개선함 토큰을 검증하고 디코드하는 과정을 공유하기 쉬움
서버에서 oauth 버튼을 불러오는 API 삭제 oauth 적용
useEffect 안쪽에서 비동기적으로 일이 질행되도록 변경함 메인 화면 로딩 속도가 개선됨
없어진 함수를 import 하는 문제
반환받은 객체가 빈 객체일때 빈객체를 false값으로 간주하고 코딩해서 생긴 에러
시간변환, axios 요청등을 다양하게 활용가능하기 때문에 utils 폴더에서 따로 관리한다.
점점 비대해지는 WorkDetail 을 유지보수하기 힘들 것 같아서 WorksDetailContainer를 만듬 -> 로직 WorkDetail -> 뷰만 담당 추후 WorkDetail을 분리하여 리렌더링되는 범위또한 조절 가능
업로드 상세 페이지는 팝업 창을 띄워 옵션을 선택하게함 tag와 user oid를 추가해햐함
업로드 선택 화면 하단에 경고 메시지란을 추가함
댓글을 5자 미만으로 썼을 때 alert 를 띄워줌
로그인 되면 헤더가 변경됨
theme.PALE_TEXT, theme.WARN_GRAY
index.tsx의 코드수가 훨씬 줄어들었다. handler 함수도 뺄수 있다는데 좀 더 고민해봐야함
Feature/jmj/upload front page
… into feature/ljh/works-detail
[Feat] 작품 디테일 페이지 디자인, 댓글
app.tsx 를 간결하게
Feature/sch/login page2
Fix - custom type을 까먹어서 죄송합니다.
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.
Release 0.2