codestates / codestates/art-ground

[Dev Log] 20211002/ 유다희

Open
#331 0 comments 0 reactions 1 assignee Claimed by @tkdkagody View on GitHub
Dev Log Error Handling Log
Dominant language
JavaScript
Stars
7
Forks
9
PR merge metrics
No merged PRs in 30d

Description

브랜치 : featues/admin_detail (어제 이어서)

## **오늘 TODO**

- [x] myinfo 수정된 정보 바로 렌더 시키기...!!! ㅠㅠ 성공!!
- [x] modify페이지에서 작가 아닌경우 작가수정란 없애기
- [x] about페이지 반응형1차 580, 480
- [ ] about 리뷰 슬라이더 만들기
- [ ] 비밀번호 수정페이지 눈알 추가
- [ ] 비밀번호 수정페이지 유효성검사 ?
- [ ] 반응형 시 세로길이 길어짐 : 탭버튼 스크롤포지션 반응형시 어떻게 바꿀 수 있나 알아보기

→ Position에 따라 스크롤탭 없애고, 스크롤탑 남겨둬보기?

## **ERROR노트**

## 1. mypage 모디파이 수정 완료 !

![Screenshot from 2021-10-04 14-50-01](https://user-images.githubusercontent.com/80307779/135809256-214a122d-05fa-473d-b524-4380496b8e16.png)

기존과 같이 1차로 수정post요청을 보내고 , 2차로 다시 정보를 받아오는 get요청을 보냈다.

랜더 문제라고 보기에는 로그인을 새로 하지 않으면 새로운 정보를 받아오지 못한다는게 정말 미스테리였다.

결국 알아낸것은, mypage요청을 보냈을때(유저인포 정보를 받아오는) 유저정보를 토큰에서 빼온것이 원인이었다. 그러니 db에 수정정보가 잘 들어갔는데도 불구하고 수정정보를 읽어오지 못한 것이었다.
왜 이부분을 이제야 떠올린걸까

```jsx

## 기존 !
const userInfo = isAuthorized(req);
if (userInfo) {
res.status(200).json({data: userInfo});
}
## db에서 찾아옴 !
const result = await users.findOne({
where: {
id,
},
});
const data = result.dataValues;
res.status(200).json({data,});
}
```

또한 setUserinfo에 수정된 값을 넣어주면, 바로 수정된 데이터가 렌더링 되는 이유는, userinfo상태값이 전역에 있기 때문이다. 따라서 유저인포처럼 전역에서 관리되지 않는 상태값의 수정사항이 바로 렌더링 되게끔 하기 위해서는 , 전역이나 렌더해야하는 데이터(컴포넌트)와 관련된 상태값이 더 상위에서 관리되어야한다. 이런문제는 아마 리덕스 등을 사용하면 없을 이슈로 보임

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.