aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNicholas Tay <nick@windblume.net>2022-03-24 15:46:48 +1100
committerNicholas Tay <nick@windblume.net>2022-03-24 15:46:48 +1100
commit716724df0fee78a8976d5255096e000af29daad1 (patch)
tree2a80906ebd310a50bc5d5a5ace5415ea00023073
parentabb8f691fe6f276205d1cea655ee9938194d1e67 (diff)
downloadforayios-716724df0fee78a8976d5255096e000af29daad1.tar.gz
forayios-716724df0fee78a8976d5255096e000af29daad1.tar.bz2
forayios-716724df0fee78a8976d5255096e000af29daad1.zip
Update LEARNING goals
-rw-r--r--LEARNING.md16
1 files changed, 14 insertions, 2 deletions
diff --git a/LEARNING.md b/LEARNING.md
index d106f2d..a01f53c 100644
--- a/LEARNING.md
+++ b/LEARNING.md
@@ -13,7 +13,19 @@
- Basically for constraints like `make.top.equalToSuperview().inset(8)`
- Test this out with accessibility options to see how reflow - image and labels may fight with each other
- [x] Rip out Storyboard entirely
-- [ ] Extract network requests into own class
+- [x] Extract network requests into own class
- Make network class generic
- See https://stackoverflow.com/questions/65576832/how-can-i-make-my-networking-class-generic-in-swift
-- [ ] The 'coordinator pattern'
+ - TODO: Name it 'xxFetcher'
+- [x] The 'coordinator pattern'
+ - Don't worry about child coordinator
+ - Decouple VCs from views
+- [ ] Custom views, separate View from VC
+ - HasCustomView, see FeedbackVC
+ - OK to have how I had it, taking in set data as string/images and not a ViewModel, since View could be reused in this way without coupling it to ViewModel, but either way works
+- [ ] Presenter
+ - VC shouldn't see PenguinItem, only the one for rendering, e.g. 'PenguinItemModel'
+ - ViewModel struct can be together with Presenter file if simple
+- [ ] Background thread
+ - DispatchQueue backgroundThread
+ - Just one place on the presenter to fire back on Main