https://codereview.appspot.com/8086045/diff/1/state/api/params/params.go#newcode340
state/api/params/params.go:340: StatusInfo string
Hmm, there may be some changes to status landing shortly. How bad would
it be for you if statuses were to move off the unit document and into
their own collection? I guess it'd be more megawatcher work, but I guess
there may be further consequences on your side?
LGTM -- maybe ping me tomorrow about status and status info?
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params. go params/ params. go (right):
File state/api/
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params. go#newcode340 params/ params. go:340: StatusInfo string
state/api/
Hmm, there may be some changes to status landing shortly. How bad would
it be for you if statuses were to move off the unit document and into
their own collection? I guess it'd be more megawatcher work, but I guess
there may be further consequences on your side?
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go params/ params_ test.go (right):
File state/api/
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go# newcode55 params/ params_ test.go: 55: Series: "series", // E.g. precise.
state/api/
On 2013/04/01 20:46:58, thumper wrote:
> Can we change the test so instead of using comments for "precise" and
> "wordpress" we instead have the test output show this?
+1
https:/ /codereview. appspot. com/8086045/