Giveth / giveth-dapp

Giveth Dapp for crowdfunding and managing donations on the ethereum blockchain.
https://beta.giveth.io
GNU General Public License v3.0
359 stars 230 forks source link

Display of "People" on the campaigns page actually displays # of donations #194

Closed bowensanders closed 6 years ago

bowensanders commented 6 years ago

Rather than give the correct number of "people" associated with a project, it seems that the number displayed as "people" is the number of donations to the project:

screen shot 2017-12-04 at 7 05 42 pm screen shot 2017-12-04 at 7 06 27 pm

and then a couple minutes later:

screen shot 2017-12-04 at 7 11 33 pm screen shot 2017-12-04 at 7 11 46 pm

Suggestion: Either change "people" to "donations" or change the code to display the number of people properly. Also possible is doing both, since it's a great undocumented feature! :)

bowensanders commented 6 years ago

I also wonder if the Donations / People issue is happening here too, on the milestones page for me:

screen shot 2017-12-05 at 9 21 10 pm

In a message to Vojtech I mentioned that the numbers for donations were incrementing strangely, as you can see it says 011, 01, 0 - which leads me to believe that there is a variable acting like a string and not an integer, making N + 1 to be "01" and then N + 1 would then become "011"....

screen shot 2017-12-05 at 9 24 34 pm

This is a pic from the campaign page. just for reference.

vojtechsimetka commented 6 years ago

I believe it is because we are adding two BigNumbets together. The BigNumbers are passed as strings to avoid precision loss.

bowensanders commented 6 years ago

I think that the top and the bottom are seperate issues. Should I split the second part (0-01-011) out to a new issue?

satyavh commented 6 years ago

You are absolutely right, these counters are wrong. Good catch.

We want to focus on the amount of people supporting the project, not the amount of donations. But this needs to be changed in feathers. I created a story for it: https://github.com/Giveth/feathers-giveth/issues/66

vojtechsimetka commented 6 years ago

The counters has been fixed in the feathers rewrite.