-
Notifications
You must be signed in to change notification settings - Fork 0
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/quests repository (1) #84
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
κ³ μνμ ¨μ΅λλ€. λ³ λ¬Έμ κ° μλ€λ©΄ νμ€νΈμͺ½μ λλλ―νλ€μ :)
.map { $0.toDomain() } | ||
.toArray() | ||
.asObservable() | ||
.asSingle() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
asObservable()
μμ΄ asSingle()
μ μλλμ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
toArrayλ₯Ό νλ κ²½μ° μλ£νμ΄ λ°λμ΄λ²λ €μ asSingleμ λ°λ‘ μ€νμ΄ μ λμ΄ asObservableλ‘ obseravaleλ‘ λ°κΎΌλ€ singleλ‘ λ°κΎΈμμ΅λλ€ π
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
μ κ° μκΈ°λ‘ toArray()μ κ²°κ³Όλ singleμ΄ λλκ±Έλ‘ κΈ°μ΅ν΄μ. (μ ννμ§ μμ§λ§..!)
λ¨Έμ§λλ©΄ νλ²λ μμλ΄μΌκ² λ€μ. λ§μ½ λ§λ€λ©΄ 34,35λ²μ€μ μμ΄λ λ μ§λ..?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
νμΈν΄λ³΄λ κ·Έλ λ€μ. λ°©κΈ κ³ μ³μ 컀λ°ν΄λ¨μ΅λλ€. toArrayμ λ°νκ°μ΄ PrimitiveSequence<SingleTrait, [Quest]>λΌμ κ³ μ³€λκ±΄λ° κ°λ₯νλ€μ κ°μ¬ν©λλ€
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
μ¬μ€ Singleμ PrimitiveSequence<SingleTrait, Element>μ νμ
μ리μ΄μ€(typealias)λΌκ³ ν©λλ€!
μ¬κΈ°κΈμ νμΈν΄λ³΄λ©΄ μ’μκ²κ°μμ:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
κ³ μνμ ¨μ΅λλ€. μλ΄€μλλ€.
|
||
import Foundation | ||
|
||
enum DateFilter { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
μ΄κ² μ°μ monthκ°μ λ°μ΄ν°λ₯Ό κ°μ Έμ¬λ νμνκ²μ΄μ§μ?
κ·Όλ° μμ§μ RealmServiceμλ§ μλ κ²μ΄μ£ ?
π Issue Number
Close #81
π μμ λ΄μ
π μμ μ ν
π 체ν¬λ¦¬μ€νΈ
π PR νΉμ΄ μ¬ν