Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FEAT: LocalStorage에 로그 적재작업 #20

Merged
merged 6 commits into from
Feb 7, 2024
Merged

FEAT: LocalStorage에 로그 적재작업 #20

merged 6 commits into from
Feb 7, 2024

Conversation

JjungminLee
Copy link
Collaborator

@JjungminLee JjungminLee commented Feb 6, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

2️⃣ 알아두시면 좋아요!

KakaoTalk_Video_2024-02-06-23-41-10.mp4
  • 영상설명 : drawer에서 홈화면 이동했을 때 로그 적재 되는 모습 / 10개 쌓이면 로컬 스토리지 비우는 로직 (화면녹화가 잘려서 됐네요 죄송합니당 ㅜㅜ)
  • useEffect로 인해 로그가 두번씩 저장되는 문제를 해결했습니다 -> 같은 로그임을 판단할때 timestamp,path,serviceName을 한꺼번에 비교합니다.
  • 로그 10개가 쌓이면 백엔드에 요청을 보내는 로직을 추가했습니다.
  • customedAxios.ts 파일을 생성했습니다.
  • 백엔드분들 에게 물어봤을때 토큰이 없다고 하셔서 토큰 작업은 따로 수행하지 않았습니다 (API는 설에 주시기로 하셨어요!)
  • .env.local파일을 로컬에 만들어주시면되는데 VITE_API_URL = "" 로 만들어주시면 될 것 같아요!

3️⃣ 추후 작업

@JjungminLee
Copy link
Collaborator Author

아 ! 이전에 올리신 PR을 안봐서 userID추가를 안했네요! 추가해둘게요!

@JjungminLee JjungminLee linked an issue Feb 6, 2024 that may be closed by this pull request
Copy link
Member

@Hanna922 Hanna922 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니당~!!!

@JjungminLee JjungminLee merged commit ed97f98 into develop Feb 7, 2024
@Hanna922 Hanna922 deleted the feat/#17 branch February 7, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 로컬스토리지에 로그 저장 [FEAT] 백엔드 API 요청 메서드 개발
2 participants