[feat]: 관리자 페이지, 관리자 상세 페이지, 공고 작성 페이지 #63
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
…Team12_FE into feature/post-write-page
- development가 아닐 때만 msw import
- 추후 admin외에 더 추가할 때 handlers 폴더 아래에 생성 - public/mockServiceWorker는 기본 설정파일
- msw 라이브러리 사용 - 각 학생 컴포넌트에 user 정보 넘김 - 하위 컴포넌트인 AuthRequest.jsx에서 nickname 출력 - id에 따라 Link로 상세페이지 이동 가능
- 추후 params에 따라 데이터 fetch
This was
linked to
issues
Oct 28, 2023
This was
unlinked from
issues
Oct 28, 2023
rktdnjs
reviewed
Oct 29, 2023
src/utils/alert.js
Outdated
Comment on lines
21
to
33
|
|
||
| export const registerMessage = { | ||
| title: '공고를 등록하시겠습니까?', | ||
| html: `정보를 알맞게 입력하셨나요?<br/> | ||
| 피커는 입력한 정보를 바탕으로 움직이게 됩니다.`, | ||
| icon: 'question', | ||
| showCancelButton: true, | ||
| reverseButtons: true, | ||
| confirmButtonColor: '#0075FF', | ||
| cancelButtonColor: '#D9D9D9', | ||
| cancelButtonText: '취소', | ||
| confirmButtonText: '확인', | ||
| }; |
rktdnjs
reviewed
Oct 29, 2023
src/index.js
Outdated
Comment on lines
6
to
10
| /* eslint import/newline-after-import: "off" */ | ||
| import worker from './mocks/worker'; | ||
| if (process.env.NODE_ENV === 'development') { | ||
| worker.start(); | ||
| } |
There was a problem hiding this comment.
MSW를 사용하기 위한 구문인가 보네요! 제가 사용하게 된다면 아마 별도로 .env 파일에 REACT_APP__NODE_ENV = "development"를 넣어줘야 하겠네요!!
There was a problem hiding this comment.
저 이거 할 때 .env 파일 따로 안건드렸어요! 별도로 효원님께서 설정해주셔야 되는 건 설치말고는 없을 거에요!
rktdnjs
reviewed
Oct 29, 2023
Comment on lines
+1
to
+14
| import { http, HttpResponse } from 'msw'; | ||
| import { REQUEST_LIST, REQUEST_DETAIL } from '../data/adminData'; | ||
|
|
||
| const adminHandlers = [ | ||
| http.get('/admin', () => { | ||
| return HttpResponse.json(REQUEST_LIST); | ||
| }), | ||
|
|
||
| http.get('/admin/auth/1', () => { | ||
| return HttpResponse.json(REQUEST_DETAIL); | ||
| }), | ||
| ]; | ||
|
|
||
| export default adminHandlers; |
There was a problem hiding this comment.
여기서 페이지별로 handler들을 따로 분리하나보네요! 저도 추후 모킹 핸들러를 추가하게 되면 따로 분리하고, 별도로 만들어 주신 index.js에서 사용하면 되겠네요 :)
|
전체적으로 깔끔하게 코드 잘 짜주신거 같아요! 고생많으셨습니다 ㅎㅎ 저는 MSW 모킹 사용하셨던 내용을 집중적으로 읽어보았는데, 어떤 흐름으로 돌아가는지는 워낙 잘 적어주셔서 이해는 어느정도 했습니다! 이번주 머지 이후로, API 연동 이전에 남는 시간에 저도 모킹을 적용시켜서 테스트 해보면 좋겠네요! 이번 주도 고생 많으셨습니다 :) 👍👍 |
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 타입을 선택해주세요)
작업 사항
관리자 페이지
/components/organisms/AuthRequest.jsx/pages/AdminPage.jsxAuthRequest.jsx에 뿌림관리자 학생 인증 상세 페이지
/components/organisms/AuthDetail.jsx/pages/AdminAuthPage.jsxAuthDetail.jsx에 뿌림msw 라이브러리 (설치 해주세요~)
mocks/data/: 임시 데이터 넣어둠mocks/handlers/admin.js: 관리자 페이지 관련 get 요청 정의mocks/handlers/index.js: 핸들러 모아서 export / 추후 다른 핸들러가 더 생긴다면 추가 가능mocks/brower.js:worker정의공고글 작성 페이지
PostWritePage.jsxIIFE로 분기관련 이슈