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

[refactor][#21] caching detail #39

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

dayo2n
Copy link
Collaborator

@dayo2n dayo2n commented Mar 17, 2024

#️⃣연관된 이슈

#21

📝작업 내용

상세정보 조회 뷰가 onAppear할 때마다 detail 요청, 편집 시작할때, 취소하고 나갈때, 편집 내용 저장할 때 전부 요청하고 있었음.
처음 조회할 때만 서버에 detail 요청 → 받은 데이터를 메모리 캐시에 담음
이후 조회할 때, 편집 취소하고 나갈 때 → 메모리 캐시에 담긴 놈 띄움
편집 내용 저장할 때 → 메모리 캐시에 담긴 기존 데이터 삭제하고 다시 요청

참고

블로그에 과정담아둠! 참고

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해 주세요
추가로 중복 코드 삭제 + 없는 리소스 이름 사용 부분 수정 완

@dayo2n dayo2n self-assigned this Mar 17, 2024
@dayo2n dayo2n linked an issue Mar 17, 2024 that may be closed by this pull request
1 task
@dayo2n dayo2n added 🐛 bug Something isn't working ✨ feat New feature or request 🏗️ refactor Refactoring or improvements labels Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working ✨ feat New feature or request 🏗️ refactor Refactoring or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] api 호출을 줄이기 위한 리팩토링
1 participant