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

Royalty toggle reversed behaviour #4189

Closed
1 task done
green-jay opened this issue Oct 27, 2022 · 2 comments · Fixed by #4656
Closed
1 task done

Royalty toggle reversed behaviour #4189

green-jay opened this issue Oct 27, 2022 · 2 comments · Fixed by #4656
Labels
A-basilisk issues related to basilisk parachain bug Something isn't working

Comments

@green-jay
Copy link

What happened?

Royalty toggle turns on royalty when it's off and vice versa

Please reproduce in steps

Mint NFT -> use royalty toggle

Expected Behavior

When I turn toggle ON, i will be able to set royalty

What browsers are you seeing the problem on?

No response

At which address did you encounter bug?

No response

Are you logged in?

No response

Which wallet you are using?

No response

Screenshots

image

Relevant log output

No response

Payment link for reward

https://kodadot.xyz/transfer/?target=%3CMy_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address%3E

Code of Conduct

  • I agree to follow this project's Code of Conduct
@green-jay green-jay added bug Something isn't working A-basilisk issues related to basilisk parachain labels Oct 27, 2022
@roiLeo
Copy link
Contributor

roiLeo commented Oct 28, 2022

Thanks for the feedback, here's how I see the feature.

Current behavior:

  • when royalties switch is on: royalties are sent to the creator address
  • when royalties switch is off: you can set an address where the royalties will be sent

it is true that it's disturbing for the user

What we need to fix:

  • make it possible for the creator to disable royalties

@green-jay
Copy link
Author

Thanks for the feedback, here's how I see the feature.

Current behavior:

  • when royalties switch is on: royalties are sent to the creator address
  • when royalties switch is off: you can set an address where the royalties will be sent

it is true that it's disturbing for the user

What we need to fix:

  • make it possible for the creator to disable royalties

Ok I see, it is a bit confusing, I would expect the royalty to be disabled when the toggle is OFF. And when it is ON to have a possibility to change the default creator address.

A simple way how to achieve disabling it would be setting royalty to 0 - that also means it could never be set again by anyone else which should be exactly what we want given there is no scenario where you would like to set royalty after ownership change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-basilisk issues related to basilisk parachain bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants