Since feedback is welcome: - Documentation for `V...
# store
b
Since feedback is welcome: • Documentation for
Validator
, I haven't really found a use for it yet. Should we be making server calls with this if we're trying to keep client/server in sync? • It was a bit unclear what
store.clear
does and realized that it only clears for the SoT doesn't do anything with the fetcher. So if you want to sync client and server you'd still need a write request. • Understanding error handling. Where should try/catches be implemented? I think the assumption is that any failures are handled upstream by store and you'll get a nice
StoreReadResponse.Error
• Adding
Input
,
Output
,
Local
,
Network
and
Response
docs wherever those are used. I tend to get them jumbled up so look to other areas where they are documented to properly use them. Just having a blanket
Local
is your SoT object everywhere the
Local
type is used would be helpful -- for each type. • When to use bookkeeper. It seems helpful for pulling data from the client, but not if we want to keep client/server in sync. So the usecase seems more for a feed rather than sync. So far with the Trails and Howl examples along with the droidcon videos and documentation I was able to piece it together. Another piece of feedback is that the name of it is very generic which makes searching for help on Google more difficult so I have to specify mobilenativefoundation with my search in order to get relevant information. Overall though, thanks for creating and open sourcing this library. I'm not trying to dog all of your efforts, this is a cool library and I'm happy I'm able to use it to solve some of my issues.
🙌 1
m
This is great. Thanks for the feedback. All makes sense. Will try to address this week
• Documentation for
Validator
, I haven't really found a use for it yet. Should we be making server calls with this if we're trying to keep client/server in sync?
Added explanation for Validator
• When to use bookkeeper. It seems helpful for pulling data from the client, but not if we want to keep client/server in sync. So the usecase seems more for a feed rather than sync.
Added explanation for Bookkeeper
🙏🏼 1
Lmk if these help
b
Oh wow, a ton of docs were added. Thank you
Big improvements in the past couple weeks. I think this will help a lot with adoption
❤️ 1