BitShares Forum

Main => General Discussion => Topic started by: wmbutler on November 26, 2015, 02:10:54 pm

Title: Bitshares Graphene-ui 2.0.151125 released
Post by: wmbutler on November 26, 2015, 02:10:54 pm
All,

The team did an epic job this week, particularly @svk. Knocked out 16 bugs and 17 enhancements:

Bugs
https://github.com/cryptonomex/graphene-ui/issues?q=milestone%3A2.0.151125+is%3Aclosed+label%3Abug

Enhancements
https://github.com/cryptonomex/graphene-ui/issues?q=milestone%3A2.0.151125+is%3Aclosed+label%3Aenhancement

Official Release Notes
https://github.com/bitshares/bitshares-2/releases/tag/v2.0.151125

Next Week's Milestone
https://github.com/cryptonomex/graphene-ui/milestones/2.0.151202

Thanks to everyone who voted for me as Project Coordinator. I'll do my best to help.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: sudo on November 26, 2015, 02:18:33 pm
still  can't save language settings
default  english
change to chinese   works
closed &reopen bts   english again!!!!!!!!!!!!!!!!!!!!!!!
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: luckybit on November 26, 2015, 02:24:46 pm
 +5% +5% +5%
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: sittingduck on November 26, 2015, 03:19:41 pm
Great work!
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: wmbutler on November 26, 2015, 03:57:31 pm
@sudo language setting saving is definitely broken on the light wallets. It's fine on the web wallet. Tracking at:

https://github.com/cryptonomex/graphene-ui/issues/544
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Pheonike on November 26, 2015, 04:33:22 pm

Don't see the back/forward buttons in web wallet.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: testz on November 26, 2015, 04:44:44 pm

Don't see the back/forward buttons in web wallet.

In web wallet you can use browser back/forward buttons.  :)
Back/forward buttons was added into light wallet.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Thom on November 26, 2015, 04:45:41 pm
@wmbutler you are knocking it out of the park with your efforts to report progress and keep the team focused.

 +5% +5% +5% +5% +5% Keep it up, you're doing great!

THANK YOU!
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: bitacer on November 26, 2015, 04:59:57 pm
The guy is a machine !
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: jakub on November 26, 2015, 06:58:47 pm
 +5%
Amazing progress.
Really well done, guys.

I've updated the roadmap to include the major things that you've done in this milestone
http://neura-sx.github.io/roadmap.xml
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Akado on November 26, 2015, 08:29:59 pm
Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: svk on November 26, 2015, 08:31:43 pm
Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.

It does, although I had a report the cancel button doesn't work.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Akado on November 26, 2015, 08:55:02 pm
Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.

It does, although I had a report the cancel button doesn't work.

Well, at least the user can be aware of his open orders, that's the most important.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: 38PTSWarrior on November 26, 2015, 08:59:40 pm
Enables this version oneself :D to claim all balances in time?
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: speedy on November 26, 2015, 09:59:32 pm
Btw I had to checkout the BitShares branch, which is 1 commit ahead of this new release, yet is titled:

Quote
a couple of small fixes before 2.0.151125 release
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: wmbutler on November 27, 2015, 02:49:54 am
Actually, @svk is the machine. If you guys want to give props, please do so for him. As far as I can tell he's doing this without any formal agreements (yet) from CNX. If he were to submit a worker proposal for his efforts, I'd fully support it.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: karnal on November 27, 2015, 08:51:04 am
Don't forget to update the download link at bitshares.org!

Also, can we have GPG signatures, or at the very least SHA256 checksums on the binaries ? (mention them when posting a new release announcement here)
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: karnal on November 27, 2015, 09:04:22 am
ATTENTION The "display total assets in chosen currency" is buggy. It only shows the totals for ONE account.

Furthermore, if an(other) account is selected, it updates, then back to the dashboard, and the "total" remains as the single total of the previously chosen account.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: svk on November 27, 2015, 06:36:20 pm
ATTENTION The "display total assets in chosen currency" is buggy. It only shows the totals for ONE account.

Furthermore, if an(other) account is selected, it updates, then back to the dashboard, and the "total" remains as the single total of the previously chosen account.

This is not a bug per se. It could of course be expanded to include all accounts that you control (and only those you control then), but right now it only includes the "current account" which is either the account you choose in the trade section, or the last account that you've viewed.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: karnal on November 28, 2015, 03:27:10 pm
I find that very unexpected behaviour. Perhaps others disagree?

There should be an easy "My assets worth in $this" overview.

Done the way it is now, it definitely shouldn't keep the last account value when the dashboard is in focus.. then we have a number that refers to a subset of the total in the context of the total, it's confusing. Indeed, for some brief seconds my blood pressure shot up as it seemed BTS had gone even lower, as I (and the 3 other enthusiasts who happened to be around at that time) looked at the equivalent amount in EUR as shown by this version.. and this was before selecting ANY account (guess it picked the oldest or something).

The idea is good, but I think it needs to be contextualized better. Perhaps a smaller font and two lines: "My assets", and "Accounts I follow" .. something like that.

Thoughts?
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Akado on November 28, 2015, 09:22:48 pm
This should be posted on the BitShares Upgrade Annoucements thread
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: svk on November 29, 2015, 12:22:35 pm
I find that very unexpected behaviour. Perhaps others disagree?

There should be an easy "My assets worth in $this" overview.

Done the way it is now, it definitely shouldn't keep the last account value when the dashboard is in focus.. then we have a number that refers to a subset of the total in the context of the total, it's confusing. Indeed, for some brief seconds my blood pressure shot up as it seemed BTS had gone even lower, as I (and the 3 other enthusiasts who happened to be around at that time) looked at the equivalent amount in EUR as shown by this version.. and this was before selecting ANY account (guess it picked the oldest or something).

The idea is good, but I think it needs to be contextualized better. Perhaps a smaller font and two lines: "My assets", and "Accounts I follow" .. something like that.

Thoughts?

I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: karnal on November 29, 2015, 07:49:16 pm
 +5%
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: thera on December 03, 2015, 12:21:34 am
Actually, @svk is the machine. If you guys want to give props, please do so for him. As far as I can tell he's doing this without any formal agreements (yet) from CNX. If he were to submit a worker proposal for his efforts, I'd fully support it.

Thank you @svk and @wmbutler!

#sharebits "svk" 100 SMILE
#sharebits "wmbutler" 100 SMILE

Feel free to share the SMILEs!  :)
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: btstip on December 03, 2015, 12:22:39 am
Hey thera, here are the results of your tips...
Curious about BtsTip? Visit us at http://sharebits.io and start tipping BTS on https://bitsharestalk.org/ today!
Source: https://bitsharestalk.org/index.php/topic,20269.msg262934/topicseen.html#msg262934
Created by hybridd (https://bitsharestalk.org/index.php?action=profile;u=40140)
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: wmbutler on December 06, 2015, 03:38:27 am
Thanks for the tips. Best to retire this thread since we have more recent upgrade announcement threads.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Thom on December 06, 2015, 06:14:40 am
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: svk on December 06, 2015, 07:20:01 am
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: Thom on December 07, 2015, 01:25:02 am
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.

I'm a bit fan of tool tips. I think almost everything should have one attached. A tool tip on that value is all that's required to explain what that number represents, and, it keeps the UI clean. I had to ask what it was, b/c I'm not sure it's right, or how it calculates the value. It doesn't seem to add up in my case and the wallet contains only 2 accounts with very few assets.
Title: Re: Bitshares Graphene-ui 2.0.151125 released
Post by: wmbutler on December 07, 2015, 04:32:30 am
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.

I'm a bit fan of tool tips. I think almost everything should have one attached. A tool tip on that value is all that's required to explain what that number represents, and, it keeps the UI clean. I had to ask what it was, b/c I'm not sure it's right, or how it calculates the value. It doesn't seem to add up in my case and the wallet contains only 2 accounts with very few assets.

It doesn't currently total assets held as collateral or open orders....but it will for v1209 !!

https://github.com/cryptonomex/graphene-ui/issues/584