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] #54 - 랭킹 리스트 뷰 비즈니스 로직 구현 #55

Merged
merged 6 commits into from
Dec 27, 2022

Conversation

L-j-h-c
Copy link
Contributor

@L-j-h-c L-j-h-c commented Dec 22, 2022

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

랭킹 리스트 뷰의 비즈니스 로직 구현했습니다. Service는 아직 서버가 구현되지 않아 더미데이터를 보내고 있습니다.

랭킹 미션 리스트 뷰로의 화면전환 로직을 구현했습니다.

추후 말풍선 디자인 변경되면 1, 2, 3위의 화면전환 로직도 구현하겠습니다.

📮 관련 이슈

@L-j-h-c L-j-h-c added Feat 새로운 기능 구현 준호🏅 labels Dec 22, 2022
@L-j-h-c L-j-h-c self-assigned this Dec 22, 2022
…om:L-j-h-c/SOPT-Stamp-iOS into feature/sopt-makers#54-RankingBusiness
Copy link
Member

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

오늘도 많이 보고 배웠습니다!
수고하셨습니다~!

@@ -23,5 +23,9 @@ public class RankingRepository {
}

extension RankingRepository: RankingRepositoryInterface {

public func fetchRankingListModel() -> AnyPublisher<[Domain.RankingModel], Error> {
return Future<[RankingModel], Error> { promise in
Copy link
Member

Choose a reason for hiding this comment

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

오우 거대한 더미 데이터.. 만드느라 수고하셨습니다..!

Copy link
Member

@devxsby devxsby left a comment

Choose a reason for hiding this comment

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

감사 선배. 🙇🏻‍♀️

Comment on lines 28 to 29
? userId!
: 1
Copy link
Member

Choose a reason for hiding this comment

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

신기 ? 하네! : 요

}

extension ChartRectangleView {
static func == (left: ChartRectangleView, right: ChartRectangleView) -> Bool {
Copy link
Member

Choose a reason for hiding this comment

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

오옹 신기하다

@@ -66,7 +66,7 @@ public extension BaseService {
// MARK: - MakeRequests

extension BaseService {
func requestObjectInCombine<T: Decodable>(_ target: API) -> AnyPublisher<T?, Error> {
func requestObjectInCombine<T: Decodable>(_ target: API) -> AnyPublisher<T, Error> {
Copy link
Member

Choose a reason for hiding this comment

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

!!

@L-j-h-c L-j-h-c merged commit 47965fc into sopt-makers:develop Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 size/L 준호🏅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 랭킹 뷰 비즈니스 로직 구현
3 participants