CoinRoster / slotmachine

fruitgame
0 stars 0 forks source link

admin page - rake transactions table #42

Closed RiskingTime closed 6 years ago

RiskingTime commented 6 years ago

see explanation #41

Patrick, if necessary, can you provide a brief list of what needs to be done with an estimated timeframe?

Patrick-Bay commented 6 years ago

It would seem that most of the desired information is already present, if not in the table then at least in the database and available via an API call. Possibly the only update here may be to fix erroneous calculations and possibly add an additional historical item if required.

I would estimate 1 hour to complete this task.

To recap:

Patrick-Bay commented 6 years ago

Updates for this issue have been pulled to the development server and are now available for review immediately.

Patrick-Bay commented 6 years ago

Thanks for the feedback via phone. So at present we're just going to leave the table as-is and you will need to refresh it in order to remove any already-transferred addresses from the table.

It should be noted that we have functionality to remove transferred accounts from the table (once the transaction is complete), just commented out for the time being:

https://github.com/CoinRoster/slotmachine/blob/7e7e3c7a0d71130a5ee76fea13577a8e8c7a277e/html/admin/js/index.js#L252-L254