Skrf 146 local storage #21

hyesung99 · 2023-10-25T04:58:53Z

📝요구사항과 구현내용

로컬 스토리지 키값 상수 저장
로컬 스토리지 관련 함수 추가 (getStorage,setStorage,deleteStorage)

✨pr포인트 & 궁금한 점

getStorage가 storage를 얻는다? 라는 느낌을 줄 수도 있을것 같아서 getFromStorage같은 변수명으로 바꿀까 고민중임다

colorkite10

수고하셨습니다! :)

colorkite10 · 2023-10-25T05:57:59Z

src/utils/localStorage.ts

+  }
+};
+
+export { getStorage, setStorage, deleteStorage };


에러메시지의 내용이나 에러처리는 임시인가요? 사용자입장에서는 내용을 이해하기 어려울 것 같습니다!

에러메시지는 개발자를 위한 내용입니다! 지금은 단순히 에러를 throw했습니다 getStorage를 UI레이어에서 호출시 에러를 처리할수 있게요..!
거기서는 사용자를 위한 에러처리가 필요할것 같네요.. toast를 띄운다던지, 에러 페이지를 보여준다던지?
아니면 커스텀 에러를 발생시켜서 공통적인 처리를 해줄수도 있습니다!

SongInjae

고생하셨습니다:)

SongInjae · 2023-10-25T07:05:04Z

src/utils/localStorage.ts

+const getStorage = <T>(key: StorageKeys): T => {
+  try {
+    const jsonValue = localStorage.getItem(key);
+    return jsonValue ? JSON.parse(jsonValue) : null;


p5: setStorage와 getStorage에서 각 Value값을 넣어줄 때, 각 value나 key가 없으면 자동으로 catch문으로 가니까 return 문을 삼항연산자로 안써도 괜찮지않을까용? 제가 잘못 알고 있는 것일 수도 있어서 궁금합니다:)

JSON.parse의 오류를 잡으려고 했는데 생각해보니 구조가 좀 잘못됐네요.. 수정하겠습니다!

여담으로 삼항연산자는 스토리지에 빈문자열이 저장되어 있을때 유효하겠네요!

아아 loacalStorage에 키값이 없으면 null이 반환되네요...! 저도 잘못알고 있었습니다ㅋㅋ

SongInjae

고생하셨습니다:)

* feat : 스토리지 키 상수 저장 * feat : 로컬스토리지 관련 함수 추가 * fix : localStorage 에러 상황 변경

* design: layout ui 적용 * fix: Layout Content margin * SKRF-146 local storage 기능 추가 (#21) * feat : 스토리지 키 상수 저장 * feat : 로컬스토리지 관련 함수 추가 * fix : localStorage 에러 상황 변경 * msw 초기설정 & 초기설정 페이지 api 구현 (#23) * chore: msw 라이브러리 설치 * feat: 초기설정 페이지 API 구현 * style: api res객체 통일 * fix: msw 서버 문제 잠시 해결 * fix: 코드리뷰 반영 * SKRF-139 design banner component (#25) * design: 배너 공통 컴포넌트 구현 * design: 배너 컴포넌트 구현 * rename: import 파일 이름 변경 * SKRF-142 design: 헤더 레이아웃 공통 컴포넌트 구현 (#20) * design: 헤더 레이아웃 공통 컴포넌트 구현 * design: 헤더 레이아웃 설정 및 children컨테이너 수정 * 코드 충돌1 * fix: 코드리뷰 반영1 * fix: 코드리뷰 반영 * fix: App.tsx react-queryclient 삭제 --------- Co-authored-by: hyesung99 <[email protected]> Co-authored-by: chaeyeon LEE <[email protected]>

hyesung99 added 2 commits October 25, 2023 13:56

feat : 스토리지 키 상수 저장

46f71ac

feat : 로컬스토리지 관련 함수 추가

0f2294b

hyesung99 self-assigned this Oct 25, 2023

colorkite10 previously approved these changes Oct 25, 2023

View reviewed changes

SongInjae previously approved these changes Oct 25, 2023

View reviewed changes

fix : localStorage 에러 상황 변경

190d87d

hyesung99 dismissed stale reviews from SongInjae and colorkite10 via 190d87d October 25, 2023 07:43

colorkite10 approved these changes Oct 25, 2023

View reviewed changes

SongInjae approved these changes Oct 25, 2023

View reviewed changes

hyesung99 merged commit 0edaa9c into main Oct 25, 2023
2 checks passed

SongInjae pushed a commit that referenced this pull request Oct 26, 2023

SKRF-146 local storage 기능 추가 (#21)

3e25c34

* feat : 스토리지 키 상수 저장 * feat : 로컬스토리지 관련 함수 추가 * fix : localStorage 에러 상황 변경

hyesung99 deleted the SKRF-146-localStorage branch October 28, 2023 01:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Skrf 146 local storage #21

Skrf 146 local storage #21

hyesung99 commented Oct 25, 2023

colorkite10 left a comment

colorkite10 Oct 25, 2023

hyesung99 Oct 25, 2023

SongInjae left a comment

SongInjae Oct 25, 2023

hyesung99 Oct 25, 2023

hyesung99 Oct 25, 2023

SongInjae left a comment

Skrf 146 local storage #21

Skrf 146 local storage #21

Conversation

hyesung99 commented Oct 25, 2023

📝요구사항과 구현내용

✨pr포인트 & 궁금한 점

colorkite10 left a comment

Choose a reason for hiding this comment

colorkite10 Oct 25, 2023

Choose a reason for hiding this comment

hyesung99 Oct 25, 2023

Choose a reason for hiding this comment

SongInjae left a comment

Choose a reason for hiding this comment

SongInjae Oct 25, 2023

Choose a reason for hiding this comment

hyesung99 Oct 25, 2023

Choose a reason for hiding this comment

hyesung99 Oct 25, 2023

Choose a reason for hiding this comment

SongInjae left a comment

Choose a reason for hiding this comment