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 OCTO logo and info #4068

Merged
merged 1 commit into from Sep 21, 2020
Merged

Add OCTO logo and info #4068

merged 1 commit into from Sep 21, 2020

Conversation

octofinance
Copy link
Contributor

Adding OctoFi logo and info now that minimum requirements of 500 holders and 1000 tx have been met.

Adding OctoFi logo and info now that minimum requirements of 500 holders and 1000 tx have been met.
@ghost
Copy link

ghost commented Sep 21, 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 4068 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.

Copy link

@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

ghost commented Sep 21, 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.

(2.01710625 BNB)

@ghost ghost added the Payment Status: Paid Fee completely paid for PR label Sep 21, 2020
@ghost ghost assigned optout21 and zachzwei Sep 21, 2020
@octofinance
Copy link
Contributor Author

octofinance commented Sep 21, 2020

Thanks team! Fantastic experience with your PR setup here. Appreciate your efforts and looking forward to integration.

Also thanks to @catenocrypt for your patience recently with doco misunderstanding. Respect.

@optout21 optout21 merged commit be373e8 into trustwallet:master Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants