change buddy's contribution view mode to daily

87kangsw · 87kangsw · May 16, 2024 · Apr 24, 2024 · Apr 24, 2024 · Apr 24, 2024
commit 45383aadb3d188fe5bc749f593dd41a68206d366
diff --git a/GitTime/Sources/CompositionRoot.swift b/GitTime/Sources/CompositionRoot.swift
@@ -79,12 +79,14 @@ final class CompositionRoot {
 														  userDefaultService: userDefaultService,
 														  realmService: realmService)
 
-			let buddyController = configureBuddyScreen(crawlerService: crawlerService,
-													   realmService: realmService,
-													   userDefaultService: userDefaultService,
-													   followService: followService,
-													   userService: userService,
-													   githubService: gitHubService)
+			let buddyController = configureBuddyScreen(
+				crawlerService: crawlerService,
+				realmService: realmService,
+				followService: followService,
+				userService: userService,
+				githubService: gitHubService,
+				keychainService: keychainService
+			)
 
 			let settingController = configureSettingScreen(authService: authService,
 														   githubService: gitHubService,
@@ -231,10 +233,10 @@ extension CompositionRoot {
 	static func configureBuddyScreen(
 		crawlerService: GitTimeCrawlerServiceType,
 		realmService: RealmServiceType,
-		userDefaultService: UserDefaultsServiceType,
 		followService: FollowServiceType,
 		userService: UserServiceType,
-		githubService: GitHubServiceType
+		githubService: GitHubServiceType,
+		keychainService: KeychainServiceType
 	) -> BuddyViewController {
 
 		var presentFollowScreen: () -> FollowViewController
@@ -245,10 +247,12 @@ extension CompositionRoot {
 			return controller
 		}
 
-		let reactor = BuddyViewReactor(crawlerService: crawlerService,
-									   realmService: realmService,
-									   userDefaultService: userDefaultService,
-									   githubService: githubService)
+		let reactor = BuddyViewReactor(
+			crawlerService: crawlerService,
+			realmService: realmService,
+			githubService: githubService,
+			keychainService: keychainService
+		)
 		let controller = BuddyViewController(reactor: reactor,
 											 presentFollowScreen: presentFollowScreen)
 		controller.title = "Buddys"

diff --git a/GitTime/Sources/Utils/Extensions/Date+Extensions.swift b/GitTime/Sources/Utils/Extensions/Date+Extensions.swift
@@ -74,3 +74,13 @@ extension Date {
 		return now.compare(afterHour) == .orderedDescending
 	}
 }
+
+extension Date {
+	static func todayStringFormatted() -> String {
+		let now = Date()
+		let format = "yyyy-MM-dd"
+		let dateFormatter = DateFormatter()
+		dateFormatter.dateFormat = format
+		return dateFormatter.string(from: now)
+	}
+}
diff --git a/GitTime/Sources/ViewControllers/Buddy/BuddyViewController.swift b/GitTime/Sources/ViewControllers/Buddy/BuddyViewController.swift
@@ -49,10 +49,7 @@ final class BuddyViewController: BaseViewController, ReactorKit.View {
 
 	// MARK: Views
 	private let addBuddyButtonItem = UIBarButtonItem(barButtonSystemItem: .add, target: nil, action: nil)
-	private let modeButton = UIButton().then {
-		$0.setImage(UIImage(systemName: "square.grid.2x2"), for: .normal)
-	}
-	lazy var modeButtonItem = UIBarButtonItem(customView: modeButton)
+
 	private let editButton = UIBarButtonItem(barButtonSystemItem: .edit, target: nil, action: nil)
 
 	private let tableView = UITableView(frame: .zero, style: .insetGrouped).then {
@@ -104,7 +101,7 @@ final class BuddyViewController: BaseViewController, ReactorKit.View {
         super.viewDidLoad()
 		self.title = "Buddys"
 		self.navigationItem.leftBarButtonItem = editButton
-		self.navigationItem.rightBarButtonItems = [addBuddyButtonItem, modeButtonItem]
+		self.navigationItem.rightBarButtonItems = [addBuddyButtonItem]
 		addNotifications()
     }
 
@@ -163,11 +160,6 @@ final class BuddyViewController: BaseViewController, ReactorKit.View {
 
 			}).disposed(by: self.disposeBag)
 
-		self.modeButton.rx.tap
-			.map { Reactor.Action.changeViewMode }
-			.bind(to: reactor.action)
-			.disposed(by: self.disposeBag)
-
 		self.editButton.rx.tap
 			.subscribe(onNext: { [weak self] _ in
 				guard let self = self else { return }
@@ -204,13 +196,6 @@ final class BuddyViewController: BaseViewController, ReactorKit.View {
 			.bind(to: reactor.action)
 			.disposed(by: self.disposeBag)
 
-		reactor.state.map { $0.viewMode }
-			.distinctUntilChanged()
-			.map { $0.systemIconName }
-			.map { UIImage(systemName: $0) }
-			.bind(to: self.modeButton.rx.image(for: .normal))
-			.disposed(by: self.disposeBag)
-
 		reactor.state.map { $0.alreadyExistUser }
 			.filterNil()
 			.subscribe(onNext: { [weak self] (exist, userName) in

diff --git a/GitTime/Sources/ViewControllers/Buddy/BuddyViewReactor.swift b/GitTime/Sources/ViewControllers/Buddy/BuddyViewReactor.swift
@@ -8,6 +8,8 @@
 
 import Foundation
 
+import GitHubKit
+
 import ReactorKit
 import RxCocoa
 import RxSwift
@@ -44,7 +46,6 @@ final class BuddyViewReactor: Reactor {
 		case checkUserExist(String?)
 		case clearCheckExist
 		case addGitHubUsername(String?)
-		case changeViewMode
 		case removeGitHubUsername(String?)
 		case toastMessage(String?)
 		case checkUpdate
@@ -55,7 +56,6 @@ final class BuddyViewReactor: Reactor {
 	enum Mutation {
 		case setBuddys([ContributionInfoObject])
 		case addBuddy(ContributionInfoObject)
-		case setViewMode(BuddyViewMode)
 		case removeBuddy(Int)
 		case setAlreadyExistUser((Bool, String)?)
 		case setToastMessage(String?)
@@ -66,7 +66,7 @@ final class BuddyViewReactor: Reactor {
 	struct State {
 		var isLoading: Bool = false
 		var isRefreshing: Bool = false
-		var viewMode: BuddyViewMode
+		var viewMode: BuddyViewMode = .daily
 		var buddys: [ContributionInfoObject] = []
 		var sections: [BuddySection] {
 			var sectionItems: [BuddySectionItem] = []
@@ -88,26 +88,24 @@ final class BuddyViewReactor: Reactor {
 
 	private let crawlerService: GitTimeCrawlerServiceType
 	private let realmService: RealmServiceType
-	private let userDefaultService: UserDefaultsServiceType
 	private let githubService: GitHubServiceType
+	fileprivate let keychainService: KeychainServiceType
 
 	let initialState: State
 
 	// MARK: Initializing
 	init(
 		crawlerService: GitTimeCrawlerServiceType,
 		realmService: RealmServiceType,
-		userDefaultService: UserDefaultsServiceType,
-		githubService: GitHubServiceType
+		githubService: GitHubServiceType,
+		keychainService: KeychainServiceType
 	) {
 		self.crawlerService = crawlerService
 		self.realmService = realmService
-		self.userDefaultService = userDefaultService
 		self.githubService = githubService
+		self.keychainService = keychainService
 
-		// let period: PeriodTypes = PeriodTypes(rawValue: userdefaultsService.value(forKey: UserDefaultsKey.period) ?? "") ?? PeriodTypes.daily
-		let viewMode = BuddyViewMode(rawValue: userDefaultService.value(forKey: UserDefaultsKey.buddyViewMode) ?? "yearly") ?? .yearly
-		initialState = State(viewMode: viewMode)
+		initialState = State()
 	}
 
 	// MARK: Mutate
@@ -128,10 +126,6 @@ final class BuddyViewReactor: Reactor {
 			let endLoading: Observable<Mutation> = .just(.setLoading(false))
 			let request = self.requestContribution(userName: userName)
 			return .concat(startLoading, request, endLoading)
-		case .changeViewMode:
-			let newViewMode: BuddyViewMode = (self.currentState.viewMode == .yearly) ? .daily : .yearly
-			self.userDefaultService.set(value: newViewMode.rawValue, forKey: UserDefaultsKey.buddyViewMode)
-			return .just(.setViewMode(newViewMode))
 		case .removeGitHubUsername(let userName):
 			guard let userName = userName else { return .empty() }
 
@@ -190,8 +184,6 @@ final class BuddyViewReactor: Reactor {
 			state.buddys = addedBuddys
 		case .setBuddys(let buddys):
 			state.buddys = buddys
-		case .setViewMode(let viewMode):
-			state.viewMode = viewMode
 		case .removeBuddy(let index):
 			var buddys = state.buddys
 			buddys.remove(at: index)
@@ -223,25 +215,52 @@ final class BuddyViewReactor: Reactor {
 			}
 	}
 
-	private func requestContribution(userName: String) -> Observable<Mutation> {
-		return self.crawlerService.fetchContributionsRawdata(userName: userName)
-			.map { response -> ContributionInfo in
-				let contributionInfo = self.parseContribution(response: response)
-				return contributionInfo
+	private func fetchContributions(userName: String) -> Observable<GraphQLResponse.UserContribution> {
+		guard let accessToken = keychainService.getAccessToken() else { return .empty() }
+
+		let githubKit = GitHubKit(config: .init(token: accessToken))
+
+		return Observable.create { observer -> Disposable in
+			async {
+				do {
+					let contribution = try await githubKit.contributions(
+						userName: userName,
+						from: Date.todayStringFormatted(),
+						to: Date.todayStringFormatted()
+					)
+					observer.onNext(contribution)
+					observer.onCompleted()
+				} catch {
+					observer.onError(error)
+				}
 			}
+
+			return Disposables.create {}
+		}
+	}
+
+	private func requestContribution(userName: String) -> Observable<Mutation> {
+
+		return self.fetchContributions(userName: userName)
+			.observe(on: MainScheduler.asyncInstance)
 			.flatMap { [weak self] contributionInfo -> Observable<Mutation> in
 				guard let self = self else { return .empty() }
-				guard self.checkUserNameIsValid(originalName: userName,
-												userName: contributionInfo.userName,
-												additionalName: contributionInfo.additionalName) == true else { return .empty() }
 
-				return self.realmService.addBuddy(userName: contributionInfo.userName,
-												  additionalName: contributionInfo.additionalName,
-												  profileURL: contributionInfo.profileImageURL,
-												  contribution: contributionInfo.contributions)
-					.map { buddy -> Mutation in
-						.addBuddy(buddy)
+				return self.realmService.addBuddy(
+					userName: contributionInfo.profile.userName ?? "",
+					additionalName: contributionInfo.profile.login,
+					profileURL: contributionInfo.profile.profileURL ?? "",
+					contribution: contributionInfo.contributions.map {
+						Contribution(
+							date: $0.date,
+							contribution: $0.contributionCount,
+							hexColor: $0.color
+						)
 					}
+				)
+				.map { buddy -> Mutation in
+						.addBuddy(buddy)
+				}
 			}
 			.catch { error -> Observable<Mutation> in
 				log.error(error.localizedDescription)

diff --git a/project.yml b/project.yml
@@ -59,7 +59,7 @@ packages:
     branch: master
   GitHubKit:
     url: https://github.com/87kangsw/GitHubKit
-    from: 1.0.1  
+    from: 1.0.2
 fileGroups:
   - GitTime/Supporting Files
 targets: