https://codereview.appspot.com/6718048/diff/1/app/views/environment.js#newcode784
app/views/environment.js:784: });
This looks fine to me, can we however be sure those are the only three
states bubbling to this layer? I know that is supposed to be the case,
but should these ever come out of sync having a fallback value here
might make sense.
LGTM
https:/ /codereview. appspot. com/6718048/ diff/1/ app/views/ environment. js environment. js (right):
File app/views/
https:/ /codereview. appspot. com/6718048/ diff/1/ app/views/ environment. js#newcode784 environment. js:784: });
app/views/
This looks fine to me, can we however be sure those are the only three
states bubbling to this layer? I know that is supposed to be the case,
but should these ever come out of sync having a fallback value here
might make sense.
Probably fine for now though
https:/ /codereview. appspot. com/6718048/