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/quests repository (1) #84

Merged
merged 3 commits into from
Dec 6, 2022
Merged

Conversation

Jeonhui
Copy link
Member

@Jeonhui Jeonhui commented Dec 5, 2022

πŸ“• Issue Number

Close #81

πŸ“™ μž‘μ—… λ‚΄μ—­

κ΅¬ν˜„ λ‚΄μš© 및 μž‘μ—… ν–ˆλ˜ λ‚΄μ—­

  • save firebase 연동
  • fetch firebase 연동
  • update firebase 연동
  • delete firebase 연동
  • deleteAll firebase 연동
  • fetch (λ‹€λ₯Έ μœ μ €) firebase 연동
  • DateFilter λ³€ν™˜ 및 뢄리

πŸ“˜ μž‘μ—… μœ ν˜•

  • μ‹ κ·œ κΈ°λŠ₯ μΆ”κ°€
  • 버그 μˆ˜μ •
  • λ¦¬νŽ™ν† λ§
  • λ¬Έμ„œ μ—…λ°μ΄νŠΈ

πŸ“‹ 체크리슀트

  • Merge ν•˜λŠ” λΈŒλžœμΉ˜κ°€ μ˜¬λ°”λ₯Έκ°€?
  • μ½”λ”©μ»¨λ²€μ…˜μ„ μ€€μˆ˜ν•˜λŠ”κ°€?
  • PRκ³Ό κ΄€λ ¨μ—†λŠ” 변경사항이 μ—†λŠ”κ°€?
  • λ‚΄ μ½”λ“œμ— λŒ€ν•œ 자기 κ²€ν† κ°€ λ˜μ—ˆλŠ”κ°€?
  • 변경사항이 νš¨κ³Όμ μ΄κ±°λ‚˜ λ™μž‘μ΄ μž‘λ™ν•œλ‹€λŠ” 것을 λ³΄μ¦ν•˜λŠ” ν…ŒμŠ€νŠΈλ₯Ό μΆ”κ°€ν•˜μ˜€λŠ”κ°€?
  • μƒˆλ‘œμš΄ ν…ŒμŠ€νŠΈμ™€ 기쑴의 ν…ŒμŠ€νŠΈκ°€ 변경사항에 λŒ€ν•΄ λ§Œμ‘±ν•˜λŠ”κ°€?

πŸ“ PR 특이 사항

전체 ν…ŒμŠ€νŒ… ν›„ 확인



Copy link
Collaborator

@jinwoong16 jinwoong16 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 +32 to +35
.map { $0.toDomain() }
.toArray()
.asObservable()
.asSingle()
Copy link
Collaborator

Choose a reason for hiding this comment

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

asObservable()없이 asSingle()은 μ•ˆλ˜λ‚˜μš”?

Copy link
Member Author

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둜 λ°”κΎΈμ—ˆμŠ΅λ‹ˆλ‹€ 😊

Copy link
Collaborator

@jinwoong16 jinwoong16 Dec 5, 2022

Choose a reason for hiding this comment

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

μ œκ°€ μ•ŒκΈ°λ‘  toArray()의 κ²°κ³ΌλŠ” single이 λ˜λŠ”κ±Έλ‘œ κΈ°μ–΅ν•΄μš”. (μ •ν™•ν•˜μ§„ μ•Šμ§€λ§Œ..!)
λ¨Έμ§€λ˜λ©΄ ν•œλ²ˆλ” μ•Œμ•„λ΄μ•Όκ² λ„€μš”. λ§Œμ•½ λ§žλ‹€λ©΄ 34,35λ²ˆμ€„μ€ 없어도 될지도..?

Copy link
Member Author

Choose a reason for hiding this comment

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

ν™•μΈν•΄λ³΄λ‹ˆ κ·Έλ ‡λ„€μš”. 방금 κ³ μ³μ„œ μ»€λ°‹ν•΄λ†¨μŠ΅λ‹ˆλ‹€. toArray의 λ°˜ν™˜κ°’μ΄ PrimitiveSequence<SingleTrait, [Quest]>λΌμ„œ 고쳀던건데 κ°€λŠ₯ν•˜λ„€μš” κ°μ‚¬ν•©λ‹ˆλ‹€

Copy link
Collaborator

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)라고 ν•©λ‹ˆλ‹€!
여기글을 확인해보면 μ’‹μ„κ²ƒκ°™μ•„μš”:)

Copy link
Collaborator

@wickedRun wickedRun left a 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 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이게 μš°μ„  month같은 데이터λ₯Ό κ°€μ Έμ˜¬λ•Œ ν•„μš”ν•œκ²ƒμ΄μ§€μš”?
근데 아직은 RealmServiceμ—λ§Œ μžˆλŠ” 것이죠?

@jinwoong16 jinwoong16 merged commit ec32e91 into develop Dec 6, 2022
@Jeonhui Jeonhui deleted the refactor/QuestsRepository branch December 6, 2022 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DefaultQuestRepository Firebase 연동
3 participants