codestates / codestates/BanThing

[Dev-Log] 2022-02-25 정윤석

Open
#185 0 comments 0 reactions 1 assignee Claimed by @slight-snow View on GitHub
devlog
Dominant language
TypeScript
Stars
0
Forks
2
PR merge metrics
No merged PRs in 30d

Description

### 오늘은 어떻게 프로젝트에 기여했나요?

- 카카오 소셜 로그인에 대한 전반적인 기능을 확인하여,
버그나 오류가 있는 부분은 수정하고 개선이 필요한 기능은 추가했습니다.
- 카카오 로그인을 했을 때, LocalStorage 를 통해 카카오 로그인 상태를 저장하도록 구현했습니다.
- 카카오로 로그인을 한 경우, 마이페이지에서 비밀번호 변경이 불가능하도록 구현했습니다.
이에 따라 카카오 로그인인지 일반 로그인인지에 따라 마이페이지가 다르게 렌더링되도록 변경하였으며,
카카오 로그인 상태에서 `수정하기` 버튼을 누를 경우 등장하는 비밀번호 변경 불가 경고 모달을 추가했습니다.
- LocalStorage 의 키 값을 통해 카카오 로그인인지, 일반 로그인인지를 구분하고
어떤 로그인인지에 따라 로그아웃이나 회원탈퇴에서 다른 axios 요청이 이뤄지도록 수정했습니다.
- 카카오 로그인을 할 때, 카카오 소셜 로그인 규정상 이메일 제공이 선택적으로 되어있으나,
저희의 주된 기능에는 반드시 아이디에 상응하는 이메일이 필요하기에 카카오 로그인을 시도할 때
반드시 이메일 제공에 동의해달라는 모달을 추가하였습니다.
- 로그인 후 바로 로그아웃을 하거나, 로그인 후 다른 페이지를 다녀와서 로그아웃을 하는 것은 가능하나,
로그인 후 새로고침을 한 다음 로그아웃을 시도할 때 정상적으로 작동하지 않는 현상을 수정하였습니다.

### 오늘의 프로젝트에서 힘든 점은 무엇인가요?
- 어떤 로그인이냐에 따라 다른 axios 요청을 보내줘야하는데,
제가 맡은 로그인, 로그아웃, 회원가입, 회원탈퇴 등의 코드에서 전부 if 문으로 경우의 수를 나눠줘야했기에
코드의 흐름을 다시 읽고 수정하는 과정에서 많이 헷갈렸던 것 같습니다 :(

### 내일은 프로젝트에 기여하기 위해 무엇을 해야 하나요?
- [x] Landing Page 에 쓰이는 일러스트의 피부색을 바꿔야합니다.
- [x] Landing Page 에 쓰이는 일러스트의 배경색을 제거해야합니다.
- [ ] Landing Page 에 쓰이는 컴포넌트들의 디자인을 수정해야합니다.
- [x] 주말입니다! 아침 출석은 없고, 10시에 맞춰서 팀원들 기상 확인하기 :3

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.