search

Kitsumon / breeding-testnet

Breeding testing issues/bugs
2 stars 0 forks source link

Multiple listing kitsu #46

Closed Mistbind closed 2 years ago

Mistbind commented 2 years ago

Prerequisites

Issue tracker is ONLY used for reporting bugs. Use stackoverflow for supporting issues.

For more information, see the CONTRIBUTING guide....

Features:

Please note by far the quickest way to get a new feature is to file a Pull Request.

We will consider your request but it may be closed if it's something we're not actively planning to work on.

Bug reports:

Expected Behavior

To be able to post a kitsu on the market only once.

Current Behavior

When posting on market a player is able to post the same kitsu Multiple times for both the different and the same price The kitsu stays in the 'owned' section so can be listed and re listed. And stays in 'my nft' section of dex so can still be bred while on the market.

Purchasing the kitsu removes all listing of that kitsu

Canceling one of the posted removes all the posted ones of that kitsu

Possible Solution

Alternatively when putting  kitsu up for auction it removes all other listing of that Kitsu from the market. Removes them from the 'owned' section from the my nft section and from the 'my nft' section of dex Ideally listing at buy now market should act the same as when putting it up for auction.

Steps to Reproduce

  1. Post a kitsu as buy now on market
  2. After it's posted go to owned kitsu
  3. Re-select the same kitsu and post it for buy now

Context (Environment)

Detailed Description

Screenshot_20220529-080615_MetaMask Screenshot_20220529-082428_MetaMask Screenshot_20220529-083758_MetaMask Screenshot_20220529-083701_MetaMask Screenshot_20220529-110638_MetaMask

Possible Implementation

JayArrowz commented 2 years ago

Cannot reproduce? Maybe just a UI issue?

Mistbind commented 2 years ago

Possibly? I'll leave the six I posted on market and see what happens, when I first discovered it, it was because someone else had done it by accident and I saw there was two of the same one on market.

On Thu, Jun 2, 2022, 4:47 PM JayArrowz @.***> wrote:

Cannot reproduce? Maybe just a UI issue?

— Reply to this email directly, view it on GitHub https://github.com/Kitsumon/breeding-testnet/issues/46#issuecomment-1145375927, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZILPDNCSFMS2CH6FPG7BA3VNETYXANCNFSM5XIMADPA . You are receiving this because you authored the thread.Message ID: @.***>

JayArrowz commented 2 years ago

Can confirm this has been fixed - will close