Skip to content

add logo for BTCu#4169

Merged
Iamdeadlyz merged 3 commits into
trustwallet:masterfrom
LBGK:master
Sep 27, 2020
Merged

add logo for BTCu#4169
Iamdeadlyz merged 3 commits into
trustwallet:masterfrom
LBGK:master

Conversation

@LBGK
Copy link
Copy Markdown
Contributor

@LBGK LBGK commented Sep 27, 2020

No description provided.

@ghost
Copy link
Copy Markdown

ghost commented Sep 27, 2020

Hi! In order to compensate for the efforts of processing PRs, we kindly ask for a contribution.
Before paying the fee, make sure new tokens fulfill the acceptance criteria (there is no fee refund in case of rejection).
See also the PR Fee FAQ.

Please pay 1000 TWT with the memo 4169 to the address bnb1tqq9llyr3dyjd559dha6z5r5etk3qfwk07m098.
Alternatively, 2 BNB is also accepted (same memo & address).

QR code: Trust | other wallet

Notes:

  • New tokens without significant usage are rejected, see criteria mentioned above.
  • One PR should be for a single project; PR's with more than 10 logos will be rejected.
  • Trust Wallet Tokens (TWT) are distributed to new users through our Referral Program.
  • Payment evaluation happens when there is activity on the PR -- write a comment if you have performed the payment.
  • When payment is detected, an Accept Review is automatically placed on the PR, which is a condition for merge. The merge is still done manually.

@Iamdeadlyz
Copy link
Copy Markdown
Contributor

Hi @LBGK

Please provide more details about this project.
https://community.trustwallet.com/t/how-to-submit-a-token-logo/3863#how-to-get-approved
You can follow this format when adding an info.json file:

{
   “name”:“Bitcoin”,
   “website”:“https://bitcoin.org”,
   “source_code”:“https://github.com/bitcoin”,
   “white_paper”:“https://bitcoin.org/bitcoin.pdf”,
   “short_description”:“Bitcoin is a cryptocurrency and worldwide payment system.“,
   “explorer”:“https://blockchain.info”,
   “research”:“https://research.binance.com/en/projects/bitcoin”
}

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fee paid, review added.

@ghost
Copy link
Copy Markdown

ghost commented Sep 27, 2020

Fee is PAID, fantastic! Thanks!

The PR will be merged soon by a maintainer, and then new/updated logos should be visible in Trust Wallet.
Please note that the app caches logos for several days, so if you want to see changes immediately, use a new device, or reinstall Trust Wallet (make sure you have a backup of all your wallets first!). See the assets FAQ.
@zachzwei & @catenocrypt, please take note.

(1000 TWT)

@ghost ghost added the Payment Status: Paid Fee completely paid for PR label Sep 27, 2020
@ghost ghost assigned optout21 and zachzwei Sep 27, 2020
@Iamdeadlyz Iamdeadlyz added Status: More Info / Change Requested Waiting for submitter to add extra info or perform changes and removed Payment Status: Requested labels Sep 27, 2020
@LBGK
Copy link
Copy Markdown
Contributor Author

LBGK commented Sep 27, 2020

does token details must be required? but we have paid for it.

@zachzwei
Copy link
Copy Markdown
Contributor

does token details must be required? but we have paid for it.

Please read this:
https://community.trustwallet.com/t/how-to-submit-a-token-logo/3863#how-to-get-approved

@LBGK
Copy link
Copy Markdown
Contributor Author

LBGK commented Sep 27, 2020

does token details must be required? but we have paid for it.

Please read this:
https://community.trustwallet.com/t/how-to-submit-a-token-logo/3863#how-to-get-approved

does new pr must be created?

@Iamdeadlyz
Copy link
Copy Markdown
Contributor

does new pr must be created?

Nope. Just commit to this pull request with the added info.json file.

@LBGK
Copy link
Copy Markdown
Contributor Author

LBGK commented Sep 27, 2020

does new pr must be created?

Nope. Just commit to this pull request with the added info.json file.

ok, I just do it , thank you

- Added the correct link for explorer and removed the incorrect source code
- Resized and compressed the logo
@Iamdeadlyz Iamdeadlyz merged commit 88be455 into trustwallet:master Sep 27, 2020
@Iamdeadlyz Iamdeadlyz removed the Status: More Info / Change Requested Waiting for submitter to add extra info or perform changes label Sep 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Payment Status: Paid Fee completely paid for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants