🔨 [리팩토링] response type 통일 [WIP]

[Junow]

closes #165 closes #50 api response type 통일작업중입니다.

@bbumjun 님이 server/common/utils.ts 에 작성하셨던 message 객체는 server/common/error.ts 에 통합되었습니다.

지금 아래와 같은 상태인데 모두 401 상태코드로 되있는것은 상의후에 통일하도록 하겠습니다.

export const CAN_NOT_BE_DONE = _he(_code.NOT_ACCEPTABLE, '요청을 완료하지 못했습니다.')

export const INVALID_GOOGLE_ID = _he(_code.UNAUTHORIZED, '계정이 유효하지 않습니다.')

export const CAN_NOT_ADD_ME = _he(_code.UNAUTHORIZED, '본인은 친구로 추가할 수 없습니다.')

export const INVALID_EMAIL = _he(_code.UNAUTHORIZED, '유효하지 않은 이메일입니다.')

export const ALREADY_EXIST_FRIEND = _he(_code.UNAUTHORIZED, '이미 추가되어 있는 친구입니다.')

export const INVALID_FRIEND_ID = _he(_code.UNAUTHORIZED, '친구의 계정이 유효하지 않습니다.')

export const ERROR_OCCURED = _he(_code.UNAUTHORIZED, '오류가 발생했습니다.')

---

8e019d9 커밋으로 confirmDelete 까지 alert action 으로 다 작업했습니다.

다음에는 SweetAlert 걷어내고 alert 컴포넌트 자체제작 하고싶네요. 디자인 통일성이 alert 부분에서 좀 안맞아서요 ㅎ..

[Junow]

@bbumjun system/profile 에서 쓰이고 있는 alert.confirmDelete 는 AlertAction 을 dispatch 하는 방식으로 바꾼다면 어떻게 할 수 있을까요? modules/alert/reducer 에 넣어도 될거같긴한데 if(confirm) 일 때 reducer 에서 다른 액션을 디스패치 하는게 맞는지 잘 모르겠네요

  const onDeleteClick = () => {
    alert.confirmDelete(name).then((confirm) => {
      if (confirm) {
        dispatch(deleteFriend(uuid))
      }
    })
  }

[bbumjun]

@bbumjun system/profile 에서 쓰이고 있는 alert.confirmDelete 는 AlertAction 을 dispatch 하는 방식으로 바꾼다면 어떻게 할 수 있을까요? modules/alert/reducer 에 넣어도 될거같긴한데 if(confirm) 일 때 reducer 에서 다른 액션을 디스패치 하는게 맞는지 잘 모르겠네요

  const onDeleteClick = () => {
    alert.confirmDelete(name).then((confirm) => {
      if (confirm) {
        dispatch(deleteFriend(uuid))
      }
    })
  }

에러 관련한 alert가 아니라 모든 alert에 대한 디스패치를 모아두는 건가요? 위같은 경우는 alert와 delete action을 분리하는것이 오히려 부자연스러운거 같기도하네요

[Junow]

@bbumjun system/profile 에서 쓰이고 있는 alert.confirmDelete 는 AlertAction 을 dispatch 하는 방식으로 바꾼다면 어떻게 할 수 있을까요? modules/alert/reducer 에 넣어도 될거같긴한데 if(confirm) 일 때 reducer 에서 다른 액션을 디스패치 하는게 맞는지 잘 모르겠네요

  const onDeleteClick = () => {
    alert.confirmDelete(name).then((confirm) => {
      if (confirm) {
        dispatch(deleteFriend(uuid))
      }
    })
  }

에러 관련한 alert가 아니라 모든 alert에 대한 디스패치를 모아두는 건가요? 위같은 경우는 alert와 delete action을 분리하는것이 오히려 부자연스러운거 같기도하네요

처음엔 모든 alert 가 에러인줄 알았는데 정리하다보니 에러가 아닌 alert 도 있더라구요 그래서 alert reducer 를 따로 모아두게 됐습니다.

말씀하신 것처럼 alert 와 delete action이 분리되는 건 아니고 onDeleteClick 이 발생했을때 alert action 을 dispatch 하고 처리를 alert reducer 에서 deleteFriend action 을 dispatch 하게 되는 모양새가 옳은지 궁금한 거였습니다.

한가지 떠오르는건 콜백으로 dispatch(deleteFriend) 를 넘겨주고 alert 에서 if(confirm) 일때 콜백을 실행시킬 수도 있겠네요. view 단에서 에러 핸들링을 정의해주는게 나으니까요

[bbumjun]

@bbumjun system/profile 에서 쓰이고 있는 alert.confirmDelete 는 AlertAction 을 dispatch 하는 방식으로 바꾼다면 어떻게 할 수 있을까요? modules/alert/reducer 에 넣어도 될거같긴한데 if(confirm) 일 때 reducer 에서 다른 액션을 디스패치 하는게 맞는지 잘 모르겠네요

  const onDeleteClick = () => {
    alert.confirmDelete(name).then((confirm) => {
      if (confirm) {
        dispatch(deleteFriend(uuid))
      }
    })
  }

에러 관련한 alert가 아니라 모든 alert에 대한 디스패치를 모아두는 건가요? 위같은 경우는 alert와 delete action을 분리하는것이 오히려 부자연스러운거 같기도하네요

처음엔 모든 alert 가 에러인줄 알았는데 정리하다보니 에러가 아닌 alert 도 있더라구요 그래서 alert reducer 를 따로 모아두게 됐습니다.

말씀하신 것처럼 alert 와 delete action이 분리되는 건 아니고 onDeleteClick 이 발생했을때 alert action 을 dispatch 하고 처리를 alert reducer 에서 deleteFriend action 을 dispatch 하게 되는 모양새가 옳은지 궁금한 거였습니다.

한가지 떠오르는건 콜백으로 dispatch(deleteFriend) 를 넘겨주고 alert 에서 if(confirm) 일때 콜백을 실행시킬 수도 있겠네요. view 단에서 에러 핸들링을 정의해주는게 나으니까요

말씀하신 방법대로 콜백으로 넘겨주면 문제될건 없을것 같네요 👍