Code Review for CS-1016

[jwelborn-sugar]

I think if this is the only time we use underscore, can we use Object.assign() and not add a dependency?

If we're going to possibly use these env options, should we go ahead and include them as template parameters?

[sugarhqi]

I included underscore mainly because we are using it for frontend, so I guess everybody is used to it and people may need it in other lambda functions.

As for platform and cliend_id, should we hard code them instead? beacuse we dont want customers to use random values?

[jwelborn-sugar]

underscore makes sense. Everyone knows it.

for client_id and platform, I think the fallback value for clientId is fine. I guess I'm just surprised the API call with an empty platform is working and not failing platform validation in the Sugar API. Did I maybe miss the value getting added somewhere?