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

Feature Request: Implement rarity-skin #100

Closed
t0bou opened this issue Oct 14, 2021 · 1 comment Β· Fixed by #116
Closed

Feature Request: Implement rarity-skin #100

t0bou opened this issue Oct 14, 2021 · 1 comment Β· Fixed by #116
Assignees
Labels
enhancement New feature or request

Comments

@t0bou
Copy link

t0bou commented Oct 14, 2021

What problem will this solve ? βœ…

Each class gets one appearance, but there is no way to personalize your look, reducing the feeling of ownership of one's summoners.


What is your solution πŸ”§

I suggest to integrate the rarity-skin open standard, managed by this contract : https://ftmscan.com/address/0xd1447fe5e70d58204946d61224643738ba54f5cc#code
It allows players to choose any NFT and assign them as avatars for their summoner. When no skin is assigned to a summoner, the current, default appearance is displayed. When the RaritySkinManager contract signals that a NFT is assigned to a summoner, the relevant image is fetched using ERC721's tokenURI() method.


Additional context πŸ“

I'm the developer of this standard, if you signal that you are interested, I offer to develop the solution and submit it to you through merge request.

@t0bou t0bou added the enhancement New feature or request label Oct 14, 2021
@Majorfi Majorfi changed the title Feature Request: Feature Request: Implement rarity-skin Oct 15, 2021
@Majorfi
Copy link
Contributor

Majorfi commented Oct 15, 2021

Thank you for this PR !
I like this standard and I would be nice to have this to Rarity Extended !
We currently have a lot of things going on but if you submit a PR for this I would be pleased to review that !

Feel free to pm me on telegram if you have some questions : @major-eth

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants