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

Add assets for 'WOJA-ac90ab' #3262

Merged

Conversation

WOJAINU
Copy link
Contributor

@WOJAINU WOJAINU commented Sep 11, 2024

No description provided.

@paulmultiversx
Copy link
Collaborator

Hi, @WOJAINU

Please provide a signature for the latest commit sha: 0ec15e9eeced98757766034c592ce2dd5661e306 which must be signed with the owner wallet address:
erd1uyh8yg9lcgxvnjqfw69dm25d92tu0mhxrw78xjjh2vuzc6dm3m6qcjd44h

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

Thanks.

@WOJAINU
Copy link
Contributor Author

WOJAINU commented Sep 12, 2024 via email

@paulmultiversx
Copy link
Collaborator

Hi, @WOJAINU

You need to paste the signature here in a new comment.

@WOJAINU
Copy link
Contributor Author

WOJAINU commented Sep 13, 2024 via email

@paulmultiversx
Copy link
Collaborator

paulmultiversx commented Sep 13, 2024

@WOJAINU the PR cannot be approved if it doesn't pass the Github checks for the signature ownership.

Please have a look at this PR as an example and see how they provided the signature signed with the owner address wallet: #3277

You must do the same using our Utils dApp: https://utils.multiversx.com/

@WOJAINU
Copy link
Contributor Author

WOJAINU commented Sep 13, 2024

{
"address": "erd1uyh8yg9lcgxvnjqfw69dm25d92tu0mhxrw78xjjh2vuzc6dm3m6qcjd44h",
"message": "0x30656331356539656563656439383735373736363033346335393263653264643536363165333036",
"signature": "0x573b1a020bbef1ca92a5543b1f5ba07e1611aab4c52f807561ee2b1ce7ddf61ec93c1eda76a6bdb35ee2eb76ef87bd0835fa4dc3a9f888c3475a9107684e9500",
"version": 1,
"signer": "ErdJS"
}

@WOJAINU
Copy link
Contributor Author

WOJAINU commented Sep 13, 2024 via email

@paulmultiversx paulmultiversx merged commit c98e11c into multiversx:master Sep 14, 2024
2 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