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.

Sign up for GitHub

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

Jump to bottom

Adding BZR token info and logo to Metamask #1102

Merged
merged 13 commits into from
Oct 16, 2022
Merged

Adding BZR token info and logo to Metamask #1102

merged 13 commits into from
Oct 16, 2022

Conversation

BazaarsBZR
Copy link
Contributor

@BazaarsBZR BazaarsBZR commented Oct 6, 2022 •
edited
Loading

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

@BazaarsBZR
Add files via upload
65a297e 
Adding BZR.png
@BazaarsBZR
Update Contract map json
43687c1 
Adding BZR contract and info 

0x8d96B4ab6C741a4C8679AE323a100d74f085BA8F
@BazaarsBZR BazaarsBZR requested review from MRabenda and a team 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?

@BazaarsBZR
Update contract-map.json
497ae99 
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

@mcmire mcmire mentioned this pull request Oct 11, 2022
Closed
@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
Reviewers

@MRabenda MRabenda MRabenda approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BazaarsBZR @MRabenda