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

Add tokenlist-extended validation #17864

Merged
merged 1 commit into from
Feb 3, 2022
Merged

Conversation

unanoc
Copy link
Contributor

@unanoc unanoc commented Feb 3, 2022

Closes #17845

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 3, 2022

PR Summary
file is not allowed: internal/file/path.go: Please revert it.
file is not allowed: internal/file/types.go: Please revert it.
file is not allowed: internal/processor/service.go: Please revert it.
file is not allowed: internal/processor/validators.go: Please revert it.

@merge-fee-bot
Copy link

merge-fee-bot bot commented Feb 3, 2022

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 1000 TWT with the memo 17864 to the address bnb1tqq9llyr3dyjd559dha6z5r5etk3qfwk07m098.
Alternatively, 5 BNB is also accepted (same memo & address).

QR code: Trust | other wallet

Notes:

  • Trust Wallet Tokens (TWT) can be obtained through our Referral Program or from DEXs/exchanges.
  • 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.
  • TWT-BEP2 is supported (Binance Chain), TWT-BEP20 version on Smart Chain is not.

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.

@unanoc unanoc merged commit 143d32c into master Feb 3, 2022
@unanoc unanoc deleted the daniel/tokenlist-validation2 branch February 3, 2022 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add validation for tokenlists
1 participant