Skip to content

Commit

Permalink
add a note about blipUrls being derived state
Browse files Browse the repository at this point in the history
  • Loading branch information
jebeck committed Feb 11, 2016
1 parent 8a0e9fb commit 9238889
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions docs/state/StateTreeGlossary.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,8 @@ The `forgotPassword` and `signUp` links are built as part of the app initializat

The `viewDataLink` is built every time the `uploadTargetUser` is chosen or changed.

**NB:** Storing these URLs as state is not ideal. Both the forgot password and sign-up URLs are essentially *derived* state - derived from a combination of route paths (e.g., `/signup`), which are constants, and a single piece of state - the server environment. The `viewDataLink` is also derived state from a combination of route paths, the server environment, and the `uploadTargetUser`. For now, we are keeping these URLs in the state tree because we do *not* represent the server environment in the state tree. We don't represent the server environment in the state tree because we have code running separately from the main application (in the Chrome App's "background page") that provides a hidden interface (for internal Tidepool use) for changing the server environment.

#### `devices`

*The `devices` property on the state tree is an object keyed by the `id` of each device (or data source) supported by the Tidepool Uploader.*
Expand Down

0 comments on commit 9238889

Please sign in to comment.