전남대_12조_픽업셔틀_7-9주차 #77
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
…feature/post-write-page
- 로그인 유무는 localStorage의 accessToken을 통해 확인 - 접근권한 체크는 localStorage의 grade를 통해 확인 - 접근권한은 해당 컴포넌트로 감싼곳에서 ProtectedRoute에 props로 requiredAuth 값을 배열형태로 제공하면 적용 - 접근권한을 따로 제공하지 않으면 로그인 유무만 판단한다고 생각하면 됨 - 로그인 + 접근권한을 둘 다 체크해야하는 곳에서는 로그인(1순위) - 접근권한(2순위)로 적용됨 - 로그인 x 인 경우에는 로그인 경고 메시지와 함께 로그인페이지로 이동 - 접근권한 x 인 경우에는 접근 거부 메시지와 함께 마이페이지로 이동(학생증 인증 권유)
- 현재는 임시 데이터로 저장하도록 구현 - 백엔드 API 연동이후 로그인 기능이 정상작동하면 accessToken, grade, username 저장하도록 변경 예정
[feat]: 공고 작성 페이지 버그 수정
[feat] : 로그인 로직 보완 & 로그인 및 접근권한 체크용 ProtectedRoute 컴포넌트 구현
[7주차] : 개발 사항 Merge(weekly-7 → develop)
…Team12_FE into feature/post-write-page
- development가 아닐 때만 msw import
- 추후 admin외에 더 추가할 때 handlers 폴더 아래에 생성 - public/mockServiceWorker는 기본 설정파일
- msw 라이브러리 사용 - 각 학생 컴포넌트에 user 정보 넘김 - 하위 컴포넌트인 AuthRequest.jsx에서 nickname 출력 - id에 따라 Link로 상세페이지 이동 가능
- 추후 params에 따라 데이터 fetch
[9주차] : 9주차 개발 코드 Merge
[9주차] : 개발 사항 Merge(weekly-9 → develop)
[9주차] : 개발 사항 Merge(develop → master)
imjasper2023
requested review from
imjasper2023
and removed request for
yoeubi
imjasper2023
reviewed
Nov 6, 2023
| /* eslint-disable */ | ||
| const Info = ({ response }) => { | ||
| // 마감시간 | ||
| let finishTime = new Date(response.finishedAt); |
imjasper2023
reviewed
Nov 6, 2023
Comment on lines
+9
to
+13
| const showMenu = (item) => { | ||
| return item.map(({ name }) => { | ||
| return <div key={name}>{name}</div>; | ||
| }); | ||
| }; |
imjasper2023
reviewed
Nov 6, 2023
| // 사실상 user 권한의 경우는 로그인 토큰 자체만으로도 검증이 가능하기 때문에, 별도로 requiredAuth prop을 넘겨줄 필요 없음 | ||
| // [admin, student] 이렇게 넘겨줄 수도 있는 경우를 감안하여 다음과 같이 처리 | ||
| // requiredAuth 중 하나라도 만족하면, 페이지 접근 권한 유효함 | ||
| const userAuth = localStorage.getItem('userAuth'); |
There was a problem hiding this comment.
userAuth에 대한 용도는 잘 파악이 안되지만 사용자 데이터를 저장하는것은 지양합니다.
imjasper2023
reviewed
Nov 6, 2023
| @@ -0,0 +1,10 @@ | |||
| const AuthDetail = ({ user }) => { | |||
There was a problem hiding this comment.
가끔 props를 다 뭉뚱그려서 하나로 퉁치는데 다 풀어서 작성부탁드려요
재사용성에서 제약사항이 생깁니다.
imjasper2023
reviewed
Nov 6, 2023
| import { BsArrowDown } from 'react-icons/bs'; | ||
| import { MdLocationPin, MdOutlineLocationOn } from 'react-icons/md'; | ||
|
|
||
| const Location = ({ response }) => { |
There was a problem hiding this comment.
Suggested change
| const Location = ({ response }) => { | |
| const Location = ({ shopName, desination }) => { |
imjasper2023
approved these changes
Nov 6, 2023
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.
PR 타입(하나 이상의 PR 타입을 선택해주세요)
전달 사항
안녕하십니까 멘토님! 이번 7-9주차 개발 진행 사항을 모아 PR 드립니다!
이번주차 멘토링 사전질문 & 개발 내용과 관련된 질문은 멘토링 전까지 하단 링크에 별도로 정리해놓도록 하겠습니다😊
개발 마지막까지 최선을 다해 끝마쳐보도록 하겠습니다..!!(화이팅)
작업 사항
7주차 개발사항
ProtectedRoute.jsx구현localStorage의 accessToken을 통해 확인localStorage의grade를 통해 확인ProtectedRoute에props로requiredAuth값을 배열형태로 제공하면 적용8주차 개발사항
유저 = 공고글 작성자여부에 따라 공고상세페이지를 달리 보여주는 부분은 개발 진행중Admin, Student, Guest페이지를 보여줄 수 있도록 분기 처리/components/organisms/AuthRequest.jsx/pages/AdminPage.jsxAdminPage에서 데이터 불러오기 → 하위AuthRequest.jsx에 뿌림/components/organisms/AuthDetail.jsx/pages/AdminAuthPage.jsxAdminAuthPage에서 데이터 불러오기→ 하위AuthDetail.jsx에 뿌림msw로id를params로 불러오는 부분이 구현이 안됨PostWritePage.jsxIIFE로 분기input에 에러메시지가 안뜨는 문제 발생 → 해결 중alert창utils로RegisterBankPage.jsxBankForm.jsx구현Swal로 경고 alert 띄움mocks/data/: 임시 데이터 넣어둠mocks/handlers/admin.js: 관리자 페이지 관련 get 요청 정의mocks/handlers/index.js: 핸들러 모아서 export / 추후 다른 핸들러가 더 생긴다면 추가 가능mocks/brower.js:worker정의9주차 개발 사항
AdminMyPageTemplate(관리자)GuestMyPageTemplate(학생)StudentMyPageTemplate(일반 유저, 학생증 인증 x 상태)PickerMatchTemplate(사용자가 매칭완료된 글을 클릭)(자신의 글 x)PickerNoMatchTemplate(사용자가 매칭전 공고글 클릭)(자신의 글 x)WriterMatchTemplate(사용자가 매칭완료된 자신의 공고글을 클릭)WriterNoMatchTemplate(사용자가 매칭전 자신의 공고글 클릭)date.js코드 수정console.log제거lastpage : false인 경우에 대해 렌더링 과정에서 버그가 있는데, 이는 API 연동을 통해 잡아야 할 것으로 보임useFormContext를 사용하여props drilling을 막으려고 함 (노력은 했는데 전이랑 똑같은 것 같기도..)msw적용npm install @hookform/error-message해주세요!)utils/dateAndTime.js에 정의)constant로 분리useMutation적용useMutation적용NaN시간 렌더링 버그Swal이 OK만 띄우는 버그div가 아닌dijv로 되어있던 부분 수정관련 PR