Gene Contains the Link set stuff for browse - this makes the gene id needlessly complex
AREMbeddedArtworkVC POTENTIALLY CREAKY HACK - Headers. Fix by adding an ORStackView to the top of the embedded VC!
ArtsyAPI the whole Browse section is pretty hacky the ARRouter needs docs! Naming of things around the concept of favouriting
Show Feed Progress dots aren't the right size Height for the hero units is based on two values when it should be one
Artwork Deleting all subviews before layout Tap gesture on artist name
Artist YOLO'd all over the switch. It's not re-usable at all anywhere else. Manual positioning for many many views.
Flow Layout Only bothered with the 2 columns
Misc ARShadowView should die
Martsy Forces http, not https
ARTextView Paragraph / line spacing is a hack
ARUserManager DRY
-- Robb feedback
Parallax: Have VC that is the root of the app, that contains a header view, and that controls buttons and nav buttons. This negates the need for a scrollcheif. Can add an ar_rootNavController etc. Only way to get smooth and catch all edge cases. E.g. the show feed headers could be a
ORStackView: leaks FLKAutoLayout be more view controller friendly ORStackViewController (deferring view will appear etc could work) addvc with title
Avoid if (! foo) bang space
Interactive transitions: don't translate to something with a header overall from the parallax, but centralizing will mean that they all can work again. perhaps artwork zoom back interactive rotation of artwork VIR could be a nice zoom via the transitions for rotation
Model layer: constants being used once. no point in the router indirection base model that can bootstrap by an ID, can have consistent API like update. Could be possible to drop promises then.
Network: cleanup the network API, like making the feeds async, it's done outside of the network model looks at libraries, look at octokit. robb advises prefixes in model classes some APIS take ID strings, other takes model objects. Could be unified if there's a base model. caching? local cache, get cache first then re-run with network layer.
Overall things hidden in things martsy: get an API that accepts a URL, it should respond with the model object or nil if noth