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

It is still possible to mint two identic images #10312

Open
prury opened this issue May 16, 2024 · 0 comments · May be fixed by #10334
Open

It is still possible to mint two identic images #10312

prury opened this issue May 16, 2024 · 0 comments · May be fixed by #10334
Assignees
Labels
A-drop-page issues related to drop page A-refactoring Refactoring stuff, make code more readable, mitigating tech debt bug Something isn't working p2 core functionality, or is affecting 60% of app

Comments

@prury
Copy link
Member

prury commented May 16, 2024

i remember that in the past we added a piece of code that would refresh the drop page after a mint was successful in order to reset the page states, but it was done as a temporary solution.

it would be interesting to refactor/change the code in a way that this could be handled better.

in the flow below you can observe that i can still mint identical images by minting one after another(using auto-teleport instead of normal mint):

twosameimages.webm

https://canary.kodadot.xyz/ahk/collection/379

image

@prury prury added bug Something isn't working p2 core functionality, or is affecting 60% of app A-refactoring Refactoring stuff, make code more readable, mitigating tech debt A-drop-page issues related to drop page labels May 16, 2024
@hassnian hassnian self-assigned this May 21, 2024
@hassnian hassnian linked a pull request May 22, 2024 that will close this issue
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-drop-page issues related to drop page A-refactoring Refactoring stuff, make code more readable, mitigating tech debt bug Something isn't working p2 core functionality, or is affecting 60% of app
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants