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.

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

Develop to Main merge #156

Merged
merged 23 commits into from
Nov 21, 2023
Merged

Develop to Main merge #156

merged 23 commits into from
Nov 21, 2023

Conversation

kkkapuq
Copy link
Collaborator

@kkkapuq kkkapuq commented Nov 21, 2023

개요

배포에 필요한 Develop to Main merge 작업입니다.

작업 사항

  • 앱 아이콘, 스플래시 이미지 변경
  • 리프레쉬 토큰 및 액세스 토큰 만료 시 로직 추가
  • github action 적용

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제
  • 배포 및 PR 관련

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

kkkapuq and others added 22 commits November 20, 2023 12:55
- 앱 아이콘, 스플래시 이미지 변경
- 로그아웃 시 캐싱된 유저 정보와 토큰들을 제거하면서 로그인 화면으로 이동
- 횡스크롤바로 게시판 목록을 앱바 바로 밑에 보여주고 게시판 목록 버튼을 누르면 해당 게시판으로 이동
- 추가적으로 API_URL 버그 부분 수정
- JwtFilter 헤더에 엑세스 토큰만 있을 때와 리프레시 토큰이 있을 때 유효성 검사를 변경
[Feat] Kernel360#148. 로그아웃 기능 및 프론트 메인 화면에 게시판 목록 추가
[Refactor] Kernel360#145. 앱 아이콘 이미지 변경
[feat] Kernel360#109 리프레시 토큰과 엑세스 토큰 유효성 확인 로직 변경
- refresh token이 만료되면 캐싱된 유저 정보와 토큰들을 삭제하고 로그인 화면으로 가게끔 변경
[Refactor] Kernel360#112. 토큰 만료됐을 때의 시나리오의 일련의 과정들이 추가 개발
- 동일한 형태의 에러 메시지를 클라이언트에 보내기 위해
예외가 발생하면 글로벌 예외처리 핸들러로 보내도록 변경
- main branch 가 push 및 pr 시 자동으로 배포되도록 처리
[feat] Kernel360#152 시큐리티 필터 단 글로벌 예외처리
@kkkapuq kkkapuq added 🌏 Deploy 배포 관련 💻BE 백엔드 📱FE 프론트엔드 labels Nov 21, 2023
@kkkapuq kkkapuq self-assigned this Nov 21, 2023
Copy link
Collaborator

@ShineCorine ShineCorine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앱을 보니 대단하시단 말밖에는..

frontend/lib/global/http/http_request.dart Outdated Show resolved Hide resolved
@kkkapuq kkkapuq mentioned this pull request Nov 21, 2023
14 tasks
kkkapuq added a commit that referenced this pull request Nov 21, 2023
[Fix] #156. 리뷰 수정사항 반영
Copy link
Collaborator

@aqrms aqrms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ShineCorine ShineCorine merged commit e4fd080 into Kernel360:main Nov 21, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻BE 백엔드 🌏 Deploy 배포 관련 📱FE 프론트엔드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants