Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[7주차] : 개발 사항 Merge(develop → master) #60

Merged
merged 13 commits into from
Oct 29, 2023
Merged

[7주차] : 개발 사항 Merge(develop → master) #60

merged 13 commits into from
Oct 29, 2023

Conversation

rktdnjs
Copy link
Contributor

@rktdnjs rktdnjs commented Oct 22, 2023

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • CSS등 UI 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

작업 사항

  • 공고 작성 페이지 일부 버그 fix
  • 로그인 & 접근권한 체크 컴포넌트 ProtectedRoute.jsx 구현
    • 로그인 유무는 localStorage의 accessToken을 통해 확인
    • 접근권한 체크는 localStorage의 grade를 통해 확인
    • 접근권한은 해당 컴포넌트로 감싼곳에서 ProtectedRoute에 props로 requiredAuth 값을 배열형태로 제공하면 적용
    • 접근권한을 따로 제공하지 않으면 로그인 유무만 판단한다고 생각하면 됨
    • 로그인 + 접근권한을 둘 다 체크해야하는 곳에서는 로그인(1순위) - 접근권한(2순위)로 적용됨
    • 로그인 x 인 경우에는 로그인 경고 메시지와 함께 로그인페이지로 이동
    • 접근권한 x 인 경우에는 접근 거부 메시지와 함께 마이페이지로 이동(학생증 인증 권유)

관련 이슈

baegyeong and others added 13 commits October 16, 2023 19:25
@baegyeong
[fix]: 요청사항은 필수항목에서 제외
467caac
@baegyeong
[fix]: 음료 갯수 배열 로직 변경
077044c
@baegyeong
[conf]: react-query 원래대로 버전 변경
cfb51fe
@rktdnjs
[conf] : react/jsx-wrap-multilines: off eslint 규칙 추가
e36c9ab
@rktdnjs
[feat] : 접근 권한 없음 경고 메시지 alert.js 추가
c2edc01
@rktdnjs
[feat] : 로그인 & 접근권한 체크 컴포넌트 ProtectedRoute.jsx 구현
d0327d8 
- 로그인 유무는 localStorage의 accessToken을 통해 확인
- 접근권한 체크는 localStorage의 grade를 통해 확인
- 접근권한은 해당 컴포넌트로 감싼곳에서 ProtectedRoute에 props로 requiredAuth 값을 배열형태로 제공하면 적용
- 접근권한을 따로 제공하지 않으면 로그인 유무만 판단한다고 생각하면 됨
- 로그인 + 접근권한을 둘 다 체크해야하는 곳에서는 로그인(1순위) - 접근권한(2순위)로 적용됨
- 로그인 x 인 경우에는 로그인 경고 메시지와 함께 로그인페이지로 이동
- 접근권한 x 인 경우에는 접근 거부 메시지와 함께 마이페이지로 이동(학생증 인증 권유)
@rktdnjs
[feat] : MyPage에 ProtectedRoute 적용
942be0b
@rktdnjs
[feat] : 로그인 이후 localStorage 데이터 처리 로직 구현
ba9f36d 
- 현재는 임시 데이터로 저장하도록 구현
- 백엔드 API 연동이후 로그인 기능이 정상작동하면 accessToken, grade, username 저장하도록 변경 예정
@rktdnjs
[refactor] : userAuth & grade 명칭 통일 -> userAuth
c6d9ef7
@rktdnjs
[refactor] : ProtectedRoute 컴포넌트 코드 리팩토링
2c9e37f
@rktdnjs
Merge pull request #54 from baegyeong/feature/post-write-page
ca13667 
[feat]: 공고 작성 페이지 버그 수정
@rktdnjs
Merge pull request #56 from rktdnjs/feature/login-page
a4c04b5 
[feat] : 로그인 로직 보완 & 로그인 및 접근권한 체크용 ProtectedRoute 컴포넌트 구현
@rktdnjs
Merge pull request #59 from Step3-kakao-tech-campus/weekly-7
93bb422 
[7주차] : 개발 사항 Merge(weekly-7 → develop)
@rktdnjs rktdnjs added the ✨ feature This feature will be developed label Oct 22, 2023
@rktdnjs rktdnjs self-assigned this Oct 22, 2023
@rktdnjs
Copy link
Contributor Author

rktdnjs commented Oct 22, 2023 •
edited
Loading 

원래는 master 브랜치에 머지 해야하나, 6주차 PR이 리뷰되지 않은 상황에서 머지할 경우 내용이 합쳐지기 때문에 
6주차 PR 리뷰 및 머지가 완료되고 나서 추후 develop → master 머지하고 master → review에  7-8주차 PR 한꺼번에 
PR하도록 하겠습니다!

9주차 PR시 전남대_12조_픽업셔틀_7-8주차 이름으로 한꺼번에 PR 하겠습니다!

@rktdnjs rktdnjs merged commit 1fee676 into master Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@rktdnjs rktdnjs

Labels
✨ feature This feature will be developed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rktdnjs @baegyeong