F-orever / planit-frontend

0 stars 3 forks source link

Feature/#23 : 이미지 파일 Firebase로 이동 및 코드 수정 #27

Closed SugarSyrup closed 9 months ago

SugarSyrup commented 9 months ago

작업내용

중점적으로 봐주었으면 하는 부분

db/repository/storage.ts : Storage에서 이미지 URL을 받아오는 함수 utils/LazyImage.tsx : Promise type의 imgURL을 img에 반영되게 수정

check📝

chodakk commented 8 months ago

로컬 이미지 파일을 제거하고 파이어베이스에 저장하게 된 이유가 궁금합니다!

그리고 R&D 과제에서는 노트북 로컬 서버에서 물리 서버로 마이그레이션한다고 되어있어서 해당 작업을 그대로 기입하기엔 앞뒤가 초큼 안맞다고 생각이 드는데 어떻게 생각하시나요?

SugarSyrup commented 8 months ago

로컬 이미지 파일을 제거하고 파이어베이스에 저장하게 된 이유가 궁금합니다!

그리고 R&D 과제에서는 노트북 로컬 서버에서 물리 서버로 마이그레이션한다고 되어있어서 해당 작업을 그대로 기입하기엔 앞뒤가 초큼 안맞다고 생각이 드는데 어떻게 생각하시나요?

이미지가 넘 많아서 파이어베이스에 옮기는 것 만으로도 프로젝트 크기가 줄어들어서 초기 렌더링 속도에 향상이 있을거라고 생각해서 파이어베이스로 저장했어요

그대로! 기입하면 안맞는거 같긴해서 파이어베이스만 빼고 서버에 저장했다고 정리해도 괜찮지 않을까요??