DemocracyEarth / old_dapp

Liquid democracy smart contract implementation
32 stars 13 forks source link

implement getter to fetch ballot creation info from Ethereum and IPFS #50

Open virgile-dev opened 6 years ago

virgile-dev commented 6 years ago

Ballot information that we need to query:

The transaction status is only needed when the user creates the ballot. I'm thinking to construct the page we don't really have to get title, author name, date of creation if we just cache them and construct the page with the info. That way the only getter we'd need would be the transaction status. What do you think @LucasIsasmendi @aecc ?

virgile-dev commented 6 years ago

The ballot page when just created:

capture d ecran 2018-04-22 a 12 48 03
AlexJupiter commented 6 years ago

If the proposed changes are implemented in https://github.com/DemocracyEarth/contracts/issues/51 I think this issue is redundant right?

aecc commented 6 years ago

Still the we need to implement fetching title, author, date from the blockchain. I'll be working on this soon

LucasIsasmendi commented 6 years ago

title, author and date are not coming from IPFS?

aecc commented 6 years ago

yes, I meant from IPFS, through the blockchain first. As I see only author name is missing. The rest, title and date seem implemented

AlexJupiter commented 6 years ago

@aecc @LucasIsasmendi there seem to be some duplicates of the points here in https://github.com/DemocracyEarth/dapp/issues/49, should this issue be closed and we just refer to #49 ?

Also, is this in "in progress", and if so, which one of you is working on this?

LucasIsasmendi commented 6 years ago

There is a pending "author name" implementation, the trick here is if we should rely on IPFS or ETH for that. But yes, we can close this because we changed the view model of this ballot detail page

aecc commented 6 years ago

@AlexJupiter @LucasIsasmendi I thought the #49 was a pure UI or UX task. There is work to do about the author unrelated to UX. @LucasIsasmendi as you already closed the issue, wanna add it somewhere else so it's not forgotten?

LucasIsasmendi commented 6 years ago

You are right @aecc , reopening this to complete pending task