Topic: Need help (and ready to pay): Building Poloniex trading tool (Read 884 times)

Your idea probably won't work... the only way to execute parts of the script is to make a function call from a cell right? So, even if you create global variables... and populate them, you'd need to have a function call in each cell that you wanted populated... that would require a minimum of 5 or 6 function calls to populate each cell:

cellB1 =getOrderNumber()
cellB2 =getTradeID()
cellB3 =getType()
cellB4 =getDate()
cellB5 =getAmount()
cellB6 =getRate()
cellB7 =getTotal()

Also, you'd have no way to know how many ResultingTrades there were... there could be multiple resulting from one buy/sell order... (ie. you want to buy 10 coins, but there is only lots of 1coin,3coin,3coin,2coin,4coin for sale... you would end up with at least 3 and possibly 5 trades to fulfil your order for 10 coins)... so you would need to multiple blocks of cells, each making function calls to get the individual values for each "resultingTrade"... there is also the possibility that there are NO resultingTrades if you put in an order that is not immediately fulfilled... so you'd probably need to wrap them all in IF() statements:

CellA2=getNumberResultingTrades()

CellB2=if($A$2>0,getTradeID(0),"")
CellC2=if($A$2>1,getTradeID(1),"")
CellD2=if($A$2>2,getTradeID(2),"")
...
Cellx2=if($A$2>x,getTradeID(x),"")
etc

Then just create "get" functions for each part of the Trade data that you want:

etc etc
and do more =IF() statements:
CellB3=if($A$2>0,getTradeType(0),"")
CellC3=if($A$2>1,getTradeType(1),"")
CellD3=if($A$2>2,getTradeType(2),"")
...
Cellx3=if($A$2>x,getTradeType(x),"")

CellB4=if($A$2>0,getTradeDate(0),"")
CellC4=if($A$2>1,getTradeDate(1),"")
CellD4=if($A$2>2,getTradeDate(2),"")
...
Cellx4=if($A$2>x,getTradeDate(x),"")

etc


Another option could be to just return the API responses as strings, and use SPLIT() functions... but I'm not sure if the keys in the JSON objects are always in the same order... so that may cause issues as well...

I dont want to use menu bcos I want sheet to place buy/sell command auto based on other cell value. With menu I can only do it manually.
So I have an idea to do that,
When the command place a BUY/SELL order succesfully, we will assign amount, date, total, rate to variables, let say a,b,c,d. Then we set other condition to write those variables back to the sheet "output". It means we dont setValue for "output" directly from other cell, the script did it. How do you think?
**Try the last time to help me on those probs to know how far we can do. And if we cant do any thing more, just tell me I will try other way to use this code. Anyway, I had a quite good result. THanks

Seems the options are:

So it looks like you'd need to return a two-dimensional array of values which would "overflow" from the cell with the function call in it and populate the neighbouring cells... which is a bit clumsy and messy.

or

you'd need to create a custom menu: https://developers.google.com/apps-script/guides/menus

I think the custom menu approach is probably the better way to go. So you'd set the values ("buy", "currencyPair", "rate" etc) in a couple of cells, then select "Execute Buy" or whatever from the menu and the function would pick up the values from the cells, make the API and then edit the cells on the output sheet as required.

This little test code creates a custom menu (you may need to adjust your permissions when you run it)... it has two options, the first "Test Output" simply simulates getting the a JSON response string from a buy order... it parses it and outputs it all onto the "output" sheet.

The 2nd option in the menu is just from the example and pops up a dialog.

When I try to write trade data to sheet "output", it says: You dont have permission to setValue

And I found this: https://stackoverflow.com/questions/15933019/google-script-setvalue-permission

They said we can not setValue for a cell from a function placed in other cell.

Can we have another way for that?

**When I try to cancel an existed order, it said "Invalid command". I dont knw why?

To output each record to individual cells... you'd need to either set each cell value in the script or return the WHOLE output as a string and setup cells to parse the output string...

I would probably do it in the script... as there is a chance that the order may be fulfullied over multiple trades:


This code assumes there is a sheet called "output"... it outputs, starting at Column B so you can put in the appropriate labels like this:


It will create as many columns worth of Trade Data as returned by the API... so if there are 3 resulting trades, you get 3 columns of data


The API states that you just pass "orderNumber" for a CANCEL... so I guess just add in a "cancel" section when creating the payload:
call would be =poloniex("cancel","1234586")


The API says that CANCEL only returns:
if it works... so I guess you'd just test if (dataAll["success"] == 1) and output whatever message you want for success or failure...

Wowwww it really works, thank you a lot!!!!!
But It only returns order number. Can we fix it to show other details? It will be very useful if it can show amount that was accept, rate, total and date, one by one in a single cell.
I have done the same with SELL,
**Also, what about CANCEL an order with an order number?

That error is just because it is attempting to parse the return JSON from Poloniex incorrectly...


This code is assuming that the "parameter" and/or "subparam" that you pass into the poloniex() function is one of the keys that comes back in the JSON return string from Poloniex... however, the example output that the API docs give for a "buy" command is:

So the "keys" are "orderNumber" and "resultingTrades"... resultingTrades has subkeys of "amount","date","rate","total","tradeID" and "type".

What has likely happened is that your BUY order has probably been placed, but your script just failed to read the result from Poloniex... as it was trying to find a key called ["BTC_ZEC"] with subkey ["0.01"] which won't exist. Check your poloniex order history and see if it got placed.

To fix the error, you'd need to put in some code that returns something that SHOULD be returned by a successful API "buy" call... Also, you should probably put some error checking in that checks to see if the API response was an error code:
I'm not sure what data from the response you actually want to display, so I just picked the "orderNumber"...



Instead of using the "named" parameters (command,parameter,subparam) listed in the function definition... I just used arguments[0], arguments[1], arguments[2], arguments[3]... and check that the "command" is "buy", so therefore there should be 4 "arguments" (0,1,2 and 3) passed in to the function.

I have made a try with your code and it seems to work bcos there is no "Too long header" error. But when I place a buy order it said: TypeError: Cannot read property "0.01" from undefined. (line 77). (Line 77:        return dataAll[parameter][subparam] }; They said the Rate is not defined.
And I did not see you add more one params in the poloniex function (still has 3 params but we are sending 4?)
**For the prob with sheet recalculation you pointed out, I will use the function with some conditional function in the sheet so I think I can control.

I've had a quick play with Google App Script, to be honest, I'm not terribly familiar with it... but it seems to be Javascript based.

A couple of things I've found wrong with your script...

This code isn't getting each character of the signature and converting to a hex string, as seems to be the intent... instead it is getting the ENTIRE signature array each time through the loop... so, I was getting "Header too long" errors... You need to change the 2nd line in the loop to:
So that it gets each character one at a time.

Secondly, as I expected, it isn't setting the parameters... so the payload ends up being "nonce=bigLongNumber&command=buy"... but it doesn't include the currencyPair, rate or amount.

So instead of this:

Try something like this:

NOTE: this is expecting the call to poloniex() function for a "buy" to be: =poloniex("buy","currencyPair",rate,amount)

for example: =poloniex("buy","BTC_ZEC",0.01,2)


Finally, I suspect that attempting to use a Google Spreadsheet may not be the best idea... it is quite possible that whenever the sheet is recalculated, that any cell you have with =poloniex() will be automatically rerun... and you could end up placing multiple buy orders unintentionally!   (or multiple sell orders if you implement that command).

Thank you for your advice. But honestly I am not really good at script, so can you help me to get this run? I would love to pay you fee for that.

It's hard to see where the appropriate values are being set... ie. "currencyPair", "rate", and "amount". Have you tried debugging the value of payloadEncoded? As this would appear to be where the values for the "currencyPair", "rate" and "amount" values are being set (I assume from cells in the spreadsheet? ):


Can you dump the value of payloadEncoded to the console and make sure that your parameters are actually being included properly...

Also, your "poloniex" function only takes 3 parameters... but you're sending 4?

function poloniex(command,parameter,subparam)
vs
=poloniex("buy","BTC_ZEC",0.01,2)

Maybe I have made  wrong post. Pls help me to delete it if I did

I used Google Apps Script and Spreadsheet to build a trading tool on Poloniex via its API. Everything seems to run well but the buy/sell command. Whenever I place an order by placing function =poloniex("buy","BTC_ZEC",0.01,2) at a cell in the sheet the API returns


All other command like get balances, deposit address...work well.
Please give me some suggestion to address this issue. Thanks a lot!
P/S: When I check the log, it said: error: Invalid command.
My code: