modulersYJ / ganoverflow-front

2 stars 2 forks source link

fix: tags null 케이스 처리 in ChatpostDraggable 핸들러 #125

Closed ABizCho closed 1 year ago

ABizCho commented 1 year ago

https://github.com/modulersYJ/ganoverflow-front/issues/119#issuecomment-1696069940 관련 resolve fix입니다.

tags필드를 nullable한 필드로 확장했는데, 저는 확장 이후 태그 부여한 포스트에 대해서만 테스트를 했었네요ㅋㅋ null케이스에 대한 고려를 안해서 발생한 에러였습니다.

변경 전 - 후 순입니다.

      loadedMeta: {
        folderId: LoadedPost.folderId,
        chatPostId: chatpost.chatPostId,
        title: LoadedPost.chatpostName,
        category: LoadedPost.categoryName?.categoryName,
        tags:  [...LoadedPost.tags] 
      },
      loadedMeta: {
        folderId: LoadedPost.folderId,
        chatPostId: chatpost.chatPostId,
        title: LoadedPost.chatpostName,
        category: LoadedPost.categoryName?.categoryName,
        tags: LoadedPost.tags ? [...LoadedPost.tags] : [],
      },
vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ganoverflow-front ✅ Ready (Inspect) Visit Preview Aug 28, 2023 5:39pm