aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to '')
-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