chore: remove old todos

Author: posva

src/index.js

@@ -26,29 +26,20 @@ function subscribeToRefs ({
  delete subs[refKey]
  })
  if (!refKeys.length) return resolve()
- // TODO check if no ref is missing
  // TODO max depth param, default to 1?
  if (++depth > 3) throw new Error('more than 5 nested refs')
 
  refKeys.forEach(refKey => {
- // check if already bound to the same ref -> skip
- // TODO reuse if already bound?
  const sub = subs[refKey]
  const ref = refs[refKey]
 
+ // unsubscribe if bound to a different ref
  if (sub) {
  if (sub.path !== ref.path) sub.unsub()
  // if has already be bound and as we always walk the objects, it will work
  else return
  }
 
- // maybe wrap the unbind function to call unbind on every child
- // const [innerObj, innerKey] = deepGetSplit(target[key], refKey)
- // if (!innerObj) {
- // console.log('=== ERROR ===')
- // console.log(data, refKey, key, innerObj, innerKey)
- // console.log('===')
- // }
  subs[refKey] = {
  unsub: subscribeToDocument({
  ref,
@@ -69,8 +60,7 @@ function bindCollection ({
  resolve,
  reject
 }) {
- // TODO wait to get all data
- // XXX support pathes? nested.obj.list
+ // TODO support pathes? nested.obj.list (walkSet)
  const array = vm[key] = []
  const originalResolve = resolve
  let isResolved
@@ -137,7 +127,6 @@ function bindCollection ({
  resolve = ({ id }) => {
  if (id in validDocs) {
  if (++count >= expectedItems) {
- // TODO use array instead?
  originalResolve(vm[key])
  // reset resolve to noop
  resolve = _ => {}
@@ -227,7 +216,6 @@ function bindDocument ({
  }
  }, reject)
 
- // TODO return a custom unbind function that unbind all refs
  return () => {
  unbind()
  unsubscribeAll(subs)