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.

Sign up for GitHub

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

Jump to bottom

[Team-13] Delete 기능 구현 & 사용자 쿠키가 반영된 API 연결하고 화면 업데이트 #202

Open
wants to merge 7 commits into
base: team-13
Choose a base branch
from
Open

[Team-13] Delete 기능 구현 & 사용자 쿠키가 반영된 API 연결하고 화면 업데이트 #202

wants to merge 7 commits into from

Conversation

Jinsujin
Copy link

📝 Tasks

  • CoreData 연결
  • 사용자쿠키값을 앱 초기화할때 받아온다
  • 사용자 쿠키값을 반영한 전체 데이터 get API 호출
  • 응답을 CoreData 에 저장 (sync)
  • 가져온 전체 데이터를 화면에 반영

1. API 로 가져온 데이터를 CoreData 에 반영

🔹 고민

  • API 에서 가져온 데이터를 앱 내부(CoreData) 와 비교해 싱크를 맞춰줘야 하는 작업
  • 현재 id 값만 비교해 중복된 id 가 CoreData 에 있으면 CoreData 에 저장하지 않는 로직입니다
  • 만약에 필드값이 하나라도 다르면 서버에서 가져온 데이터를 CoreData 에 반영해야 합니다
  • 필드값 하나하나를 비교해 다른 데이터인지 판별하는 방법은 비효율적이라는 생각이 듭니다
  • 싱크를 어떻게 맞춰야 할지 고민입니다

2. Delete API 연결

🔹 작업

  • REST-API 중 삭제를 요청하는 코드를 작성하였습니다.
  • 화면에서 Delete를 요청할 수 있도록 UITableView Delegate를 사용하였습니다.
  • 그 외 스토리보드 작업을 진행하였습니다.

Jinsujin and others added 6 commits April 15, 2022 07:18
@Jinsujin
feat: CoreData 연결 (#23)
12b19e0 
SceneDelegate 에서 Root View Controller 에 메서드를 통해 주입
CoreData 연결 확인함
@Jinsujin
feat: PersistenceProvider 싱글톤 제거, CardDataMiddleWare 에 주입 (#23)
11b7a04
@Jinsujin
feat: 유저 쿠키를 사용해 전체 데이터를 불러오는 API 호출, 뷰에 반영 (#23)
fc6d4ed 
1. 실제 사용자 쿠키를 앱 실행시(AppDelegate.didFinishLaunching) 얻어 온다
2. Debug 용 api 호출이 아닌, 사용자 쿠키를 담은 API 를 호출한다
3. 응답으로 받은 전체 카드 데이터를 MiddleWare 에 반영
4. 화면 업데이트
화면에서 뿌려줄 모델을 CardData -> Card 로 변경
Card 모델은 앱 내부에서만 사용될 모델의 역할
CardData 모델은 API를 이용할때만 사용
@SangHwi-Back
Merge pull request #53 from Jinsujin/feature-23
deca528 
[Feat] 사용자 쿠키가 반영된 API 연결 & 화면 업데이트
@SangHwi-Back
[Feat] Delete API 연결
7799e99
@Jinsujin
Merge pull request #55 from SangHwi-Back/feature-delete
16f00e7 
[Feat] Delete API 연결
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mienne mienne Awaiting requested review from mienne

Assignees

@Jinsujin Jinsujin

@SangHwi-Back SangHwi-Back

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jinsujin @SangHwi-Back