Skip to content

Create btc.gs - #88

Merged
grant merged 2 commits into
masterfrom
grant-sheets-btc
Dec 21, 2018
Merged

Create btc.gs#88
grant merged 2 commits into
masterfrom
grant-sheets-btc

Conversation

@grant

@grant grant commented Dec 16, 2018

Copy link
Copy Markdown
Contributor

Adds the BTC custom function demo from Grant's Apps Script talks

Adds the BTC custom function demo from Grant's Apps Script talks
* @return The value of BTC in USD at the date. From Coinbase's API
* @customfunction
*/
function BTC(date) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is the style for Apps Script? should this be btc or is BTC okay?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Google Sheets uses upper case for function names. Thus BTC.

Comment thread sheets/customFunctions/btc.gs Outdated
* limitations under the License.
*/

var COINBASE_API_TOKEN = ''; // TODO

Copy link
Copy Markdown
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 README or instructions on how to get this?

@grant
grant merged commit bb01146 into master Dec 21, 2018
@grant
grant deleted the grant-sheets-btc branch December 21, 2018 00:02
brentchang pushed a commit to brentchang/apps-script-samples that referenced this pull request Dec 3, 2023
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