Skip to content

Main#65

Open
seong-ji-sue wants to merge 6 commits into
developfrom
main
Open

Main#65
seong-ji-sue wants to merge 6 commits into
developfrom
main

Conversation

@seong-ji-sue

Copy link
Copy Markdown
Contributor

markdown

PR 체크리스트

PR을 올렸다면 아래 사항은 반드시 지켜주세요.

  • Jira에 맞는 적절한 브랜치를 생성했습니다
  • PR 우측 Labels에서 적절한 라벨을 부착하였습니다.
  • Commit 메세지 규칙에 맞게 작성했습니다.
  • 제가 의도한 파일들과 수정 사항들만 커밋이 된 것을 확인 하였습니다.
  • git rebase와 squash를 했고 커밋 수가 하나인 것을 확인 했습니다.
  • 기능에대한 오류가 없는것을 확인했습니다
  • 브라우저 콘솔상 에러가 없는것을 확인했습니다.
  • npm start로 구현한 화면 혹은 기능을 확인했습니다
  • 코드 리뷰 사항을 모두 반영하였습니다.
  • 리뷰어에 2명 할당했습니다

마크다운에서 체크를 하기 위해서는 [x] 이렇게 수정하시면 됩니다. [x ], [ x ] 가 아니라 정확하게 [x]여야 합니다. 혹은 마우스 클릭으로 체크해주세요.

구현한것들 및 간단한 설명

  • [ ]
  • [ ]
  • [ ]

구현 화면

(여기에 구현 화면 스크린샷을 첨
부해주세요)

@seong-ji-sue seong-ji-sue self-assigned this Apr 17, 2022
seong-ji-sue and others added 4 commits April 17, 2022 14:30
* Fix : 회원가입 실패시 알람 구현, 이메일 구글 google.com => gmail.com으로 변경 (#66)

* Fix : 햄버거 메뉴 골드, 실버 리스트페이지 라우팅 추가 (#67)

* Fix : 오늘 뭐먹지 포인터 css 삭제 (#68)

* [feat, fix] 마일리지 받아오는 url 추가, 마이페이지 마일리지 적용, 리뷰 이미지 등록 주소 변경 및 리다이렉트 부분 수정, 히스토리 인자 관련 부분 삭제 => /login으로 location.replace 하는 로직으로 변경, 디테일 페이지 진입시 로그인 체크 로직 제거, 토큰 체크 로직 변경 (#70)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants