Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove action buttons on expired offer #5378

Closed
exezbcz opened this issue Mar 25, 2023 · 5 comments · Fixed by #5381
Closed

Remove action buttons on expired offer #5378

exezbcz opened this issue Mar 25, 2023 · 5 comments · Fixed by #5381
Assignees
Labels
A-item p3 non-core, affecting less than 40% UX first Improvement for UX

Comments

@exezbcz
Copy link
Member

exezbcz commented Mar 25, 2023

image

  • The cancel and accept button should not be visible in case of expired offer
@exezbcz exezbcz added UX first Improvement for UX p3 non-core, affecting less than 40% A-item labels Mar 25, 2023
@prachi00 prachi00 self-assigned this Mar 25, 2023
@kodabot
Copy link
Collaborator

kodabot commented Mar 25, 2023

ASSIGNED - @prachi00 🔒 LOCKED -> Monday, March 27th 2023, 09:56:06 UTC -> 36 hours

prachi00 added a commit that referenced this issue Mar 25, 2023
@prachi00 prachi00 mentioned this issue Mar 25, 2023
17 tasks
@roiLeo
Copy link
Contributor

roiLeo commented Mar 27, 2023

@exezbcz this is done on purposes to let user "CANCEL" offer even when expired because this offer balance is frozen until it's accepted or removed. (ref #3582, #766)

But "Accept" button shouldn't be allowed here.

@yangwao
Copy link
Member

yangwao commented Mar 27, 2023

this offer balance is frozen until it's accepted or removed.

I think it's balance is locked there, but from bid author and can be withdraw anytime iirc.

What "CANCEL" would trigger on-chain for funds which are not owned by user?

Maybe @vikiival can bring some light, because I think only action you can do is to accept bid, you can't do anything else with incoming bid.

CANCEL issues some sort of on-chain message to bidder or how will see it?

Other hand I would be surprised if Basilisk has this?

@vikiival
Copy link
Member

vikiival commented Mar 27, 2023

As @roiLeo pointed out correctly offer can be canceled by offer maker or current NFT owner

What is happening on on-chain side is calling marketplace.withdrawOffer(collectionId, tokenId, makerAddress)

Pallet then checks if creator of tx is one of them

https://github.com/galacticcouncil/Basilisk-node/blob/master/pallets/marketplace/src/lib.rs

L236

If owner cancels a bid, there is currently no notification for the user, but can be implemented via subsquid

vikiival added a commit that referenced this issue Mar 28, 2023
@exezbcz
Copy link
Member Author

exezbcz commented Mar 28, 2023

If owner cancels a bid, there is currently no notification for the user, but can be implemented via subsquid

yes! I forgot to add it - is it possible to make like counter for the offer instead of cancel? That would be maybe more useful than canceling it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-item p3 non-core, affecting less than 40% UX first Improvement for UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants