codestates / codestates/BanThing
[Dev-Log] 2022-02-24 정윤석
- Dominant language
- TypeScript
- Stars
- 0
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
### 오늘은 어떻게 프로젝트에 기여했나요?
- 아이디나 닉네임을 작성한 후 중복확인을 누르면 '사용가능한 아이디/닉네임입니다.' 라는 확인 문구를 노출시키도록 구현했습니다.
이전의 유효성 검사를 통해 이미 존재하는지의 여부만 판단할 수 있었는데,
'이미 존재하는 아이디/닉네임입니다.' 라는 경고 문구가 나왔다가 사용가능한 아이디/닉네임을 입력하고 중복확인 버튼을 눌렀을 때
경고 문구가 사라지고 아무것도 노출되지 않아 사용이 가능하다는 것인지 아닌지 애매한 부분이 있었기에 이에 따라 문구를 추가했습니다.
- 회원탈퇴 후 Landing Page 로 리다이렉트됨과 동시에 로그아웃 처리되도록 구현했습니다.
- 로그아웃시, 회원탈퇴시 cookie 가 삭제되도록 구현했습니다.
- 비밀번호 변경에 대한 모달창을 추가하였습니다.
단순히 비밀번호가 변경되는 것만으로는 비밀번호가 변경됐는지 안 됐는지 확인하기 어렵다는 판단 하에,
비밀번호 변경이 완료되었다는 문구를 담은 모달을 제작하였습니다.
- 비밀번호 변경 관련 또 다른 경우의 수를 발견하고, 수정하였습니다.
변경할 비밀번호(이하, `변경`)과 비밀번호 확인(이하, `확인`)을 일치시킨 다음 `변경`을 바꾸면 `변경`과 `확인`은 다른 값을 가지게 돼죠.
하지만 그 상태에서 비밀번호 수정하기 버튼을 누르면 비밀번호가 변경되는 현상을 확인했습니다.
즉, `변경`과 `확인`이 일치하지 않음에도 비밀번호가 수정되는 현상이 발견되어 해당 경우의 수에 대한 조건문 작성해두었습니다.
- Header 의 깨짐 현상을 전반적으로 수정하였습니다.
Chrome 에서는 문제 없었으나, FireFox 혹은 Whale 브라우저에서 헤더가 화면을 벗어나는 것을 확인했고,
padding 속성이 일부 사이즈에서 깨지는 것을 확인했습니다.
CSS 파일 수정을 통해 위의 사항들 모두 개선하였습니다.
- 로그인 모달에서 아이디와 비밀번호를 입력하고 `enter` 키만으로 로그인이 되도록 구현했습니다.
( 같은 팀의 승빈님께서 직접 이용해보시다가 말씀해주신 개선점입니다. 감사합니다 :3 )
### 오늘의 프로젝트에서 힘든 점은 무엇인가요?
- server 에서 cookie 에 accessToken 을 담아서 보내줌에도 불구하고,
client 상에서 cookie 에 accessToken 이 정상적으로 담기지 않아 여러모로 어려움을 겪었습니다.
로그인 상태이냐 아니냐를 cookie 로 판단하도록 수정하였기에, 이 부분이 해결되지 않으면
회원탈퇴를 하거나 로그아웃을 해도 로그인 상태가 유지되는 것과 같이 로그인 상태가 엉망이 되기 때문입니다.
- 다행히 `withCredentials: true` 를 통해 이를 해결할 수 있었으며,
코드에 작성해두고도 해당 코드를 `headers` 내에 넣어둔 것이 문제였습니다 :(
해결해서는 다행이지만 이틀 가까이 이 문제를 해결하지 못했었는데 의외로 해법이 간단해서 그런지 조금 허무한 감이 있네요.
```javascript
axios
.delete(`${process.env.NEXT_PUBLIC_SERVER_ENDPOINT}/users/signout`, {
headers: {
Authorization: `Bearer ${accessToken}`,
'Content-Type': 'application/json',
},
withCredentials: true,
})
```
### 내일은 프로젝트에 기여하기 위해 무엇을 해야 하나요?
- [x] 카카오 소셜로그인일 경우 My Page 에서 비밀번호 변경 기능을 비활성화 해야합니다.
- [x] 카카오 소셜로그인에 대한 기능 구현을 완성시켜야합니다.
- [ ] 9시에 팀원들 출석 알림, 이후 기상 체크하기
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.