Skip to content

Commit

Permalink
Remove obsolete comments
Browse files Browse the repository at this point in the history
  • Loading branch information
0x80 committed Jun 3, 2020
1 parent 0be9299 commit 2cde8fb
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
3 changes: 0 additions & 3 deletions src/collection.ts
Expand Up @@ -225,9 +225,6 @@ export class ObservableCollection<T> {
* After the first promise has been resolved we want subsequent calls to
* ready() to immediately return with the available data. Ready is only
* meant to be used for initial data fetching
*
* @TODO change document to data maybe because data is observable so it
* won't get stale.
*/
this.readyPromise = Promise.resolve(this.docs)
}
Expand Down
8 changes: 0 additions & 8 deletions src/document.ts
Expand Up @@ -238,9 +238,6 @@ export class ObservableDocument<T> {
* After the first promise has been resolved we want subsequent calls to
* ready() to immediately return with the available data. Ready is only
* meant to be used for initial data fetching
*
* @TODO change document to data maybe because data is observable so it
* won't get stale.
*/
this.readyPromise = Promise.resolve(this.data)
}
Expand All @@ -259,11 +256,6 @@ export class ObservableDocument<T> {
return;
}

// if (!this._ref) {// this.changeLoadingState(false);

// throw Error("Can not fetch data on document with undefined ref");
// }

this.logDebug("Fetch initial data");

/**
Expand Down

0 comments on commit 2cde8fb

Please sign in to comment.