Skip to content

Conversation

@julien51
Copy link
Contributor

@julien51 julien51 commented Dec 7, 2024

Adding the UP token on Base

@merge-fee-bot
Copy link

merge-fee-bot bot commented Dec 7, 2024

PR Summary

file is not allowed: blockchains/base/tokenlist.json: Please revert it.
Token in PR: BASE 0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187
Symbol: UP decimals: 18
Links: Logo Info Website ExplorerFromInfo ExplorerFromID
Tags: defi, nft, governance
Token check error:
❌ explorer field incorrect: should be standard https://basescan.org/token/0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187, not https://basescan.org/token/0xAC1Bd2486aAf3B5C0fc3Fd868558b082a531B2B4
❌ invalid value 'https://twitter.com/unlockprotocol' for twitter link url, allowed only with prefix: https://x.com/
❌ number of holders not checked: please, check it manually

@merge-fee-bot
Copy link

merge-fee-bot bot commented Dec 7, 2024

Hi! In order to compensate for the efforts of processing PRs, we kindly ask for a contribution.
💀 As there is no refund, before paying the fee, make sure new tokens fulfill the minimum circulation and other acceptance criteria.
See also the PR Fee FAQ.

Please pay 500 TWT to the BEP20 (Binance Smartchain) address 0x1DB8C94Af50dfC1b9a55394a8e7746E389792ef1.
Alternatively, 2.5 BNB is also accepted (same address).

QR code: Trust | other wallet

TWT Acquiring:

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.
  • Payment evaluation happens automatically, but with a few minutes delay. When payment is detected, an Accept Review is automatically placed on the PR, which is a condition for merge.
  • Evaluating the PR is done manually, and it is merged only if all conditions are satisfied.
    There will be a fee to process this request. None of it goes to the developers.
    Before paying the fee, make sure new tokens fulfill the minimum circulation and other acceptance criteria.
    If you are paying TWT for the submission, this will be burned automatically. There will be no refunds.

Copy link

@merge-fee-bot merge-fee-bot bot left a comment

Choose a reason for hiding this comment

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

Fee is PAID, fantastic! Thanks! Review added.

The PR will be evaluated soon by a maintainer, and if merged, the new logos should be visible in Trust Wallet.
Please note that the app caches logos (up to several days; if you want to see changes immediately, use a new installation). See the assets FAQ.
@Cryptocool1, @cryptomanz1: please take note.

(500.00 TWT)

@merge-fee-bot merge-fee-bot bot added the Payment Status: Paid Fee completely paid for PR label Dec 7, 2024
@merge-fee-bot
Copy link

merge-fee-bot bot commented Dec 7, 2024

500.00 TWT have been successfully burned.

@Cryptocool1
Copy link
Contributor

In order for the pull request to be approved, the submission has to meet several important requirements. For this request, it does not meet the following:

  • at least 10000 token holders
  • at least 15000 transactions

To know more about the token approval guidelines, please check this article: How to Submit Token Logo.

Feel free to re-open pull request once you meet those requirements.

@julien51
Copy link
Contributor Author

julien51 commented Dec 9, 2024

Thanks! I will re-open in due time!

@julien51
Copy link
Contributor Author

Hello @Cryptocool1 I believe token now has the required/expected number of holders.
Should I re-open?

@julien51
Copy link
Contributor Author

Hey @Cryptocool1 How can we re-open this one?

@julien51
Copy link
Contributor Author

Or maybe @cryptomanz1 ?

@Cryptocool1 Cryptocool1 reopened this Feb 26, 2025
@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 26, 2025

PR Summary

file is not allowed: blockchains/base/tokenlist.json: Please revert it.
Token in PR: BASE 0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187
Symbol: UP decimals: 18
Links: Logo Info Website Explorer
Tags: defi, nft, governance
Token check error:
❌ number of holders not checked: please, check it manually

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 26, 2025

Hi! In order to compensate for the efforts of processing PRs, we kindly ask for a contribution.
💀 As there is no refund, before paying the fee, make sure new tokens fulfill the minimum circulation and other acceptance criteria.
See also the PR Fee FAQ.

Please pay 500 TWT to the BEP20 (Binance Smartchain) address 0x1DB8C94Af50dfC1b9a55394a8e7746E389792ef1.
Alternatively, 2.5 BNB is also accepted (same address).

QR code: Trust | other wallet

TWT Acquiring:

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.
  • Payment evaluation happens automatically, but with a few minutes delay. When payment is detected, an Accept Review is automatically placed on the PR, which is a condition for merge.
  • Evaluating the PR is done manually, and it is merged only if all conditions are satisfied.
    There will be a fee to process this request. None of it goes to the developers.
    Before paying the fee, make sure new tokens fulfill the minimum circulation and other acceptance criteria.
    If you are paying TWT for the submission, this will be burned automatically. There will be no refunds.

@Cryptocool1
Copy link
Contributor

Please fix

  • logo size to 256/256
  • twitter -> x

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 26, 2025

PR Summary

file is not allowed: blockchains/base/tokenlist.json: Please revert it.
Token in PR: BASE 0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187
Symbol: UP decimals: 18
Links: Logo Info Website Explorer
Tags: defi, nft, governance
Token check error:
❌ invalid value for links.name filed, allowed only: medium, facebook, blog, forum, source_code, github, telegram_news, docs, discord, youtube, coinmarketcap, whitepaper, twitter, coingecko, telegram, reddit
❌ number of holders not checked: please, check it manually

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 26, 2025

PR Summary

file is not allowed: blockchains/base/tokenlist.json: Please revert it.
Token in PR: BASE 0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187
Symbol: UP decimals: 18
Links: Logo Info Website Explorer
Tags: defi, nft, governance
Token check error:
❌ invalid value for links.name filed, allowed only: telegram, reddit, coingecko, forum, source_code, github, telegram_news, medium, facebook, blog, discord, youtube, docs, whitepaper, twitter, coinmarketcap
❌ number of holders not checked: please, check it manually

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 26, 2025

PR Summary

file is not allowed: blockchains/base/tokenlist.json: Please revert it.
Token in PR: BASE 0xaC27fa800955849d6D17cC8952Ba9dD6EAA66187
Symbol: UP decimals: 18
Links: Logo Info Website Explorer
Tags: defi, nft, governance
Token check error:
❌ invalid value for links.name filed, allowed only: blog, forum, source_code, github, telegram_news, medium, facebook, discord, youtube, docs, whitepaper, twitter, coinmarketcap, telegram, reddit, coingecko
❌ number of holders not checked: please, check it manually

@Cryptocool1
Copy link
Contributor

Thank you

@Cryptocool1 Cryptocool1 merged commit 504e845 into trustwallet:master Feb 26, 2025
3 checks passed
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.

3 participants