Skip to content

Adding BZR token info and logo to Metamask#1102

Merged
MRabenda merged 13 commits intoMetaMask:masterfrom
BazaarsBZR:master
Oct 16, 2022
Merged

Adding BZR token info and logo to Metamask#1102
MRabenda merged 13 commits intoMetaMask:masterfrom
BazaarsBZR:master

Conversation

@BazaarsBZR
Copy link
Contributor

@BazaarsBZR BazaarsBZR commented Oct 6, 2022

Bazaars website:
https://bazaars.io/

BZR token contract address

0x8d96B4ab6C741a4C8679AE323a100d74f085BA8F

Token tracker Etherscan
:
https://etherscan.io/token/0x8d96b4ab6c741a4c8679ae323a100d74f085ba8f

Coingecko:
https://www.coingecko.com/en/coins/bazaars

Whitepaper:
https://bazaars.io/wp-content/uploads/2022/06/Bazaars.io-Whitepaper-v2.pdf

Security Audit:
https://bazaars.io/wp-content/uploads/2022/06/audit.pdf

Adding BZR.png
Adding BZR contract and info 

0x8d96B4ab6C741a4C8679AE323a100d74f085BA8F
@BazaarsBZR BazaarsBZR requested review from a team and MRabenda as code owners October 6, 2022 07:05
@MRabenda
Copy link
Collaborator

MRabenda commented Oct 7, 2022

SVG image is required - can you please replace PNG one?

Changed BZR logo from png to svg
@BazaarsBZR
Copy link
Contributor Author

SVG image is required - can you please replace PNG one?

Also updated the contract-map.json

Thank you

@MRabenda
Copy link
Collaborator

@bazaars-project - SVG image should be vectorized, sharp and regular should be ~ 5kB. Could you improve your icon especially since you have so a nice one :)?

@BazaarsBZR
Copy link
Contributor Author

@bazaars-project - SVG image should be vectorized, sharp and regular should be ~ 5kB. Could you improve your icon especially since you have so a nice one :)?

Done

@BazaarsBZR
Copy link
Contributor Author

Do we need to do anything else ?

@MRabenda
Copy link
Collaborator

For some reason tests are failing for your PR @bazaars-project can you run npm tests on your branch an make sure that everything is fine?

@BazaarsBZR
Copy link
Contributor Author

Is it fine now?

@MRabenda
Copy link
Collaborator

@bazaars-project you have to move image from main folder to images is why tests are failing

@BazaarsBZR
Copy link
Contributor Author

@bazaars-project you have to move image from main folder to images is why tests are failing

Done, thank you

@BazaarsBZR
Copy link
Contributor Author

Could we merge please

@MRabenda MRabenda merged commit 0f19548 into MetaMask:master Oct 16, 2022
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