Skip to content

Conversation

@kyo-ago
Copy link
Contributor

@kyo-ago kyo-ago commented Dec 7, 2017

No description provided.

Copy link
Contributor

@erickoledadevrel erickoledadevrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this sample! A few small comments.

var CLIENT_SECRET = '...';

/**
* Authorizes and makes a request to the Harvest API.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the API name in this comment.

* Authorizes and makes a request to the Harvest API.
*/
function run() {
var service = getService();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This library uses 2-space tabs throughout. Please update this sample to match.

.setTokenHeaders({
'Authorization': 'Basic ' + Utilities.base64Encode(CLIENT_ID + ':' + CLIENT_SECRET)
})
.setTokenPayloadHandler(function(tokenPayload) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment that explains why this is needed?

@kyo-ago
Copy link
Contributor Author

kyo-ago commented Dec 8, 2017

Thanks for review!
And I fixed.

@erickoledadevrel erickoledadevrel merged commit 736477b into googleworkspace:master Dec 8, 2017
@erickoledadevrel
Copy link
Contributor

Looks great, thanks!

@kyo-ago kyo-ago deleted the patch-1 branch December 9, 2017 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants