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

Branding DMXT-c47816 #3299

Closed
wants to merge 12 commits into from
Closed

Branding DMXT-c47816 #3299

wants to merge 12 commits into from

Conversation

DecentralizedMxToken
Copy link
Contributor

Add assets for DecentralizedMxToken

paulmultiversx
paulmultiversx previously approved these changes Sep 23, 2024
@paulmultiversx
Copy link
Collaborator

paulmultiversx commented Sep 23, 2024

Hi, @DecentralizedMxToken

We cannot approve this PR because it is against the official MX Assets Guidelines regarding the trademarks.

'logo does not resemble MultiversX/Mobile Wallet logos or other trademarked visual identities'

So, please eliminate the MVX logo from your logo and we will be able to proceed.

Thank you for understanding.

@DecentralizedMxToken
Copy link
Contributor Author

Hi @paulmultiversx
I've removed the MVX logo from my design as per your request. Please check and let me know if we can proceed or if further adjustments are needed. Thank you for your guidance on this.

@paulmultiversx
Copy link
Collaborator

Hi, @DecentralizedMxToken

Yes, the logo is OK now but you have 4 files in your PR now and you only need 3 files, the ones in your Tokens directory.
Please delete the .vscode so we can proceed with the merging of this PR.

Thanks.

@DecentralizedMxToken
Copy link
Contributor Author

@paulmultiversx
Ok. .vscode file deleted

@paulmultiversx
Copy link
Collaborator

@DecentralizedMxToken - There are still 4 files in your PR. I recommend you close this one and start a fresh one if you don't manage to fix it.

@DecentralizedMxToken
Copy link
Contributor Author

Trying again.

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