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

Staff 1b295e asset #4015

Merged
merged 4 commits into from
Feb 10, 2025
Merged

Conversation

SharloKK
Copy link
Contributor

@SharloKK SharloKK commented Feb 9, 2025

Willing to add branding for our NFT collection which is held by a smart contract

paulmultiversx
paulmultiversx previously approved these changes Feb 10, 2025
@paulmultiversx
Copy link
Collaborator

Hi, @SharloKK

Your branch is behind our master. Please make sure you have all our updates in your forked repository so we can approve & merge the PR.


Thanks.

@SharloKK
Copy link
Contributor Author

Hi, I just updated the branch on my side with your master. Thanks

@paulmultiversx
Copy link
Collaborator

@SharloKK try again please.

@SharloKK
Copy link
Contributor Author

Just did

@paulmultiversx
Copy link
Collaborator

@SharloKK

Your SVG is not a square: 52.916668 x 52.916706

logo.svg (must be in a square format)

logo.png (must have a resolution of 200x200 pixels)

max logo size 100KB

Please re-upload it properly and after this, please provide a signature for the latest commit sha: 48c59324d44054703305c9ae27688a0fb9048317 which must be signed with the owner wallet address:
erd10dq2d89xvv2lea2hxemgje6w04eyge67h85ku96kfkvwd3mtgcmsndmgg4

Feel free to use the Sign Message feature from the Utils dApp and paste the signature here in a comment: https://utils.multiversx.com/

Step-by-step guide:

Step 1: Copy latest commit sha
Step 2: Open: https://utils.multiversx.com/
Step 3: Log in with your owner address wallet
Step 4: Click the 'Sign Message' feature
Step 5: Paste the commit sha in the first column and hit sign
Step 6: Copy-paste the provided output signature here in a new comment

Thanks.

@SharloKK
Copy link
Contributor Author

{
"message": "34386335393332346434343035343730333330356339616532373638386130666239303438333137",
"signature": "2bcbf47a2b8201758790e5e10c004b5c7b3553e7f2684e32fad8b1a25791b6c34e8f57a6d06c66e2e4cd559376c77253b72f5692f4815570cb5122dd328a3604",
"address": "erd10dq2d89xvv2lea2hxemgje6w04eyge67h85ku96kfkvwd3mtgcmsndmgg4",
"version": 1,
"signer": "sdk-js"
}

@paulmultiversx paulmultiversx merged commit 781e579 into multiversx:master Feb 10, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants