dap-ps / discover-v2

Discover all your favourite Distributed Applications, ranked in a provably fair and transparent way
Mozilla Public License 2.0
3 stars 0 forks source link

Editing DApps creates a new DB record #16

Open andytudhope opened 3 years ago

andytudhope commented 3 years ago

There are two inter-related issues here:

  1. When I edit a DApp, I have to provide a new name. This is because new edits create a new DB record. This is not how it should work. I should be able to edit an existing DApp without changing the name (most updates are for the logo, followed by the description and -very occasionally - the url).
  2. Do not create new DB records for edited DApps. It should simply update the current record with the new metadata hash sent to the contract in the edit tx, and update the relevant fields in the DB itself.
andytudhope commented 3 years ago

@RyRy79261 the most important of the three fixes that need to happen, but likely not a one night thing :wink:

If you need STT to work on dev.dap.ps you can always claim directly from the contract via etherscan, using the generateTokens method.