MikaLendingBot icon indicating copy to clipboard operation
MikaLendingBot copied to clipboard

New Bootstrap-UI

Open tamaskan opened this issue 7 years ago • 42 comments

Updated JQuery Included Standard Roboto-Font Custom Date-Format via http://momentjs.com/ Coin-Icon-Font via https://labs.allienworks.net/icons/cryptocoins/ Added Total Account-Value and Value per Coin Different collapsible Sections and Filters Progressbars and visual Min-Coin-Warning Increased max. Update-Interval to 3600

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [X] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [N/A] I have read CONTRIBUTING.md
  • [N/A] I fully understand Github Flow.
  • [X] My code adheres to the code style of this project.
  • [N/A] I have updated the documentation in /docs if I have changed the config, arguments, logic in how the bot works, or anything that understandably needs a documentation change.
  • [N/A] I have updated the config file accordingly if my change requires a new configuration setting or changes an existing one.
  • [N/A] I have tested the bot with no issues for 24 continuous hours. If issues were experienced, they have been patched and tested again.

tamaskan avatar May 24 '17 01:05 tamaskan

General comment, use spaces (not tabs), keep things correctly aligned.

rnevet avatar May 24 '17 09:05 rnevet

Do we need all the binary files? It's generaly a bad idea to commit binaries to git w/o annex or something similar.

laxdog avatar May 25 '17 06:05 laxdog

I haven't found a hosted version of the cryptocoin-font, but it could be reduced to a single .woff2-file

tamaskan avatar May 25 '17 19:05 tamaskan

'Show the amount of earned Coins since first UI-Start (Stored in localstorage)'

Not needed, we are going to have complete lending history in this branch: https://github.com/BitBotFactory/poloniexlendingbot/pull/349

Please correct tabs as it's really hard to review the code this way.

rnevet avatar May 25 '17 20:05 rnevet

While I find displaying the opened loans interesting, displaying the cancel loan events is useless as this happens on every bot cycle and doesn't actually bring any information.

rnevet avatar May 25 '17 21:05 rnevet

image

rnevet avatar May 25 '17 21:05 rnevet

The filtering buttons are ignored when a reload happens. :)

rnevet avatar May 25 '17 21:05 rnevet

I like the overall design, however I think many of the features should be discussed.

In general I would prefer to have first design change and then separate PRs for features... this will also make it easier to review and approve.

rnevet avatar May 25 '17 21:05 rnevet

The creators of this bot are free to use only the non-controversial parts and open issues for discussions :-)

tamaskan avatar May 25 '17 22:05 tamaskan

The design changes look really nice!

Hiding a bunch of stuff under accordions makes this design less practical though. I like to be able to view all the stats upon a glance. If I have to tap to open accordions, it takes time and effort. Arguably not much, but enough that it makes the old design more useful at this stage.

It gets really annoying when the bot refreshes the page and you lose all the accordion states! :(

kingo55 avatar May 28 '17 08:05 kingo55

I can change that behavior, thanks for the feedback :-)

tamaskan avatar May 28 '17 20:05 tamaskan

https://github.com/dutu/poloLender has a clean interface too, if you want a different approach

tamaskan avatar Jun 22 '17 15:06 tamaskan

How about using prettyFloat instead of printFloat ? :)

rnevet avatar Jun 23 '17 09:06 rnevet

No problem, the function-description is quite vague ^^

tamaskan avatar Jun 23 '17 20:06 tamaskan

Well it prints a pretty float with a specified accuracy. i.e. accuracy 2: 1.02332341 -> 1.0234 4556.1233 -> 456.12 1.00 -> 1 1.10 -> 1.1 etc...

rnevet avatar Jun 24 '17 08:06 rnevet

Hi, with your branch i always receive the "DataTables warning: table id={id} - Requested unknown parameter '{parameter}' for row {row-index}, column{column-index}`" error and my "Opened Loan" tabs is empty even if i have opened loan.

The error raise up on "mydatatable.fnAddData(dataSet);" invokation.

LeaderGL avatar Jun 24 '17 11:06 LeaderGL

interesting, never experienced this problem. how does your botlog.json look like ?

tamaskan avatar Jun 24 '17 20:06 tamaskan

at fail time my botlog is like that (i've removed my coin amount): {"last_status": "Lent: [0.xxxxxxxx BTC @ 0.1462%] ", "last_update": "2017-06-22 22:44:27", "log": ["2017-06-22 22:44:08 Using custom mindailyrate 0.100% for BTC", "2017-06-22 22:44:14 Not lending BTC due to rate below 0.1561%"], "outputCurrency": {"currency": "BTC", "highestBid": "1"}, "raw_data": {"BTC": {"averageLendingRate": "0.1461733644848254994269884326", "lentSum": "0.xxxxxxxx", "maxToLend": "0.xxxxxxxx", "totalCoins": "0.xxxxxxxx"}}}

you can solve the issue using code from emilam: https://github.com/emilam/poloniexlendingbot/commit/8ac76573651855b43ab193dcf46e1e16144ac704 and use all current active loans to fill the datatable

LeaderGL avatar Jun 26 '17 05:06 LeaderGL

If there is no active Loan, i would run into the Problem again, so i just added a Check :-) It would be nice if emilam could push the py-Changes, so i can adapt my Code.

tamaskan avatar Jun 26 '17 06:06 tamaskan

btw, I made a JSFiddle for prettyFloat https://jsfiddle.net/mut72zcp/

rnevet avatar Jun 26 '17 08:06 rnevet

@tamaskan how about making this an alternate theme based on: https://github.com/BitBotFactory/poloniexlendingbot/commit/86c3521f65df2ef41bd41a62c5b34c45c644cd81 ?

rnevet avatar Aug 01 '17 23:08 rnevet

some friends and i will work on an alternative frontend soon, but probably on my fork. It really needs an overhaul. I wanna try if vue.js could work and maybe replace bootstrap

mchl18 avatar Aug 01 '17 23:08 mchl18

@mchl18 have you seen the option to create alternate themes?

rnevet avatar Aug 01 '17 23:08 rnevet

@rnevet sure, that is not a problem. @mchl18 awesome :-)

tamaskan avatar Aug 01 '17 23:08 tamaskan

also wanna try and write a config interface...wip

mchl18 avatar Aug 01 '17 23:08 mchl18

@rnevet no i have not. I just fiddled with some css overlay but i think i want a complete overhaul

mchl18 avatar Aug 01 '17 23:08 mchl18

thought about a configuration-interface, but the main problem is to keep it up to date

tamaskan avatar Aug 01 '17 23:08 tamaskan

i have some ideas and will try it on my fork

mchl18 avatar Aug 01 '17 23:08 mchl18

great :-) do you need help ?

tamaskan avatar Aug 01 '17 23:08 tamaskan

I havent had time to set it up, wanna meet with some friends to discuss, if I run into questions i will let you know

mchl18 avatar Aug 01 '17 23:08 mchl18