themindhouse / cryptocanvas Goto Github PK

As a user, I want to see all canvas' transaction history. History should be placed in the details page of canvas and include:

• History of initial bidding: who won the initial bidding, and all bids placed for canvas.
• History of all finalized transactions, so when the canvas has changed the owner. Technically, it means whenever function acceptBuyOffer or acceptSellOffer is called.
• History of all made sell/buy offers. To be discussed if we want that.

Simple layout, just make it look good on mobile. Hide all controls relating to interacting with the blockchain

Account & Transactions doesn't change when I change MetaMask account. It actually seems that it shares all the information across all the MetaMask's accounts. It may be related to #2.

Example: I set all the canvas' pixels using Account 1. Then I switch to Account 2, and I still see transactions called Set pixel to.... It remains the same even if I reload the page.

When Metamask account is changed, transactions are still sent from the old one.

Suggestion: what if we reload the page each time MetaMask changes account? It should fix most of the problems of that kind.

Transaction list is not 100% consistent with its texts. There is text: Set pixel to #215 on Canvas 1, and then Buy Offer on Canvas # 1. Let's always prefix canvas' number with # character.

As a user, I want to see even more details about an account. This issue is kind of extension of #14.
Extra details to be included:

• Information about all made buy offers
• Information about all made sell offers
• List of all ever sold items
• List of all ever bought items
• All spent ethers
• All earned ethers

As a user, I want to clearly see in the website's UI if my transaction is pending. It would be good to add some more loaders, like in the setting pixel transaction. That includes:

• Add a loader to my account balance when there is pending withdrawal transaction.
• Add a loader when to canvas details window when there is a get reward transaction pending.
• Add a loader to initial bidding view, when there is post a bid transaction pending.

Note: if that issue is time-consuming let's move it out of the Release 1.0 milestone.

As a user, I want to see more details about an account in account details view. It includes:

• Add a list of owned canvases.
• Add a list of initial biddings in which given user participates. It's enough to just show a list of biddings where given user has the highest bid.
• Account balance section should additionally contain a number of owned canvases.
• Pixels Painted section should be more detailed and list all the canvases that user placed a pixel on. For instance: _Canvas # 1: 10 pixels, Canvas # 13: 1 pixel.

Add some analytics system. It doesn't have to be anything fancy. It's totally fine if it only tracks where are our users coming from - if it's a direct link, Facebook campaign, or Reddit post.

Transaction List modal looks a bit messy for me. What I'd do is:

• Redesign it a bit. Make all the text justified, or something. We can talk about it. Nothing fancy nor time-consuming. I don't want you to spend too much time on this.
• For completed and failed transaction it'd be awesome to have information how much gas has been used. I think that this is a very important point.

As a user, I want to easily find canvases that are put for sale. Let's do it in a fairly simple way: let's add a cart icon next to the canvases that are available for sale.

You can use new function for that: getCanvasesWithSellOffer(bool includePrivateOffers).

It's not possible to close pixel details modal. It happens when I'm in finished canvas details + I'm an owner of a canvas. See the screenshot below.

There seems to be something wrong with emitted events.
When accepting Sell offer, from and to addresses are the same.
When accepting Buy offer by the owner, the amount is 0 and to address is 0x0

Zooming in/out incorrectly changes pixel info window position.
To reproduce: go to an active canvas. Open pixel info window by clicking on any pixel. Zoom in/out.

Screenshot:

As a user, each time I send a transaction to the Ethereum blockchain, I want to be informed if it completed or failed.

Edit: I created that issue because it was in your ToDoist, but actually it looks like it's done. I think the box in the bottom right corner is just what we need. Please close it if you agree with that.

When I set a pixel loader shows up. The problem is that it's barely visible. Let's change its color. You can see the screenshot below.

I've been wondering if we should display Paint Pixel Transaction sent modal. Setting pixel is an operation that will be probably done a lot of times. Now, to do that, a user has to click 5 times.

Wouldn't it be better, if we display some kind of Toast/BottomBar notification that doesn't block a user from further interactions?

Let's discuss it here, and if needed create an appropriate issue.

As a user, I want to see info if I have been already rewarded for painting pixels. Right now, the button to get a reward is disabled. It'd be way better if there was an info like this: You've been already paid a reward for painting 🎉

As a user, I want to see a well-scaled gas limit. Right now it's as high as 3.000.000 which suggests our transactions are way more expensive than they really are. According to my gas calculator, it should be enough to set a limit to 150.000. Below I paste functions' gas cost.

Gas costs for a canvas of 4096 pixels:

Everywhere we show the amount in ETH we should display amount in USD, converted by the latest rate

And as a developer I want transaction history table to be consistent :P Therefore I need to add a canvas owner address to the "from" field of the CanvasOfferedForSale (Sell Offer) and CanvasNoLongerForSale (Sell Offer Cancelled) events.

Please add an indexed field from to those events, a field to in case a sell offer was for a specified address, and a field amount