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

Rebranding Genesis Lab validator to Stakeflow #26118

Merged
merged 13 commits into from
Sep 5, 2023
Merged

Rebranding Genesis Lab validator to Stakeflow #26118

merged 13 commits into from
Sep 5, 2023

Conversation

SergeyMitk
Copy link
Contributor

Info and logo changes, added providers.

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

PR Summary

File blockchains/terra/validators/assets/terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx/logo.png is being deleted. Files should not be deleted in a PR. (Deprecated tokens should be deactivated only.)

Validators check
failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

❌ Asset 'terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx' (terra) not found in list.json
failed to get logo: unsuccessful status code: 404

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

failed to get logo dimensions: failed to decode config: png: invalid format: not a PNG file

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

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 with the memo 26118 to the TWT-BEP2 (Binance Chain) address bnb158fgd782we4tz9vwwcj9pmmxvxhl6z46lfm7c8.
Alternatively, 5 BNB is also accepted (same memo & 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.

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

PR Summary

File blockchains/terra/validators/assets/terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx/logo.png is being deleted. Files should not be deleted in a PR. (Deprecated tokens should be deactivated only.)

Validators check

❌ Asset 'terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx' (terra) not found in list.json
failed to get logo: unsuccessful status code: 404

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 22, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 23, 2023

@SergeyMitk, kind reminder: please pay fee: 500 TWT with the memo 26118 to the address bnb158fgd782we4tz9vwwcj9pmmxvxhl6z46lfm7c8.
Alternatively, 5 BNB is also accepted (same memo & address).

QR code: Trust | other wallet
See the Pull Request Fee FAQ.

merge-fee-bot[bot]
merge-fee-bot bot approved these changes Aug 23, 2023
View reviewed changes
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 Aug 23, 2023
@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 23, 2023

500.00 TWT have been successfully burned.

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 23, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 25, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 25, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 26, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Aug 29, 2023

PR Summary

Files OK: 31

Validators check

@merge-fee-bot
Copy link

merge-fee-bot bot commented Sep 5, 2023

PR Summary

Files OK: 31

Validators check

❌ Asset 'umeevaloper1mkv4myppz2w0d3nr2dh9fv5kcwaaqnzfvnm6qf' (umee) not found in list.json

@merge-fee-bot
Copy link

merge-fee-bot bot commented Sep 5, 2023

PR Summary

File blockchains/terra/validators/assets/terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx/logo.png is being deleted. Files should not be deleted in a PR. (Deprecated tokens should be deactivated only.)

Validators check

❌ Asset 'terravaloper1gh7wpfpsjrqnash5uc84z4njt95y9g5nh3uqzx' (terra) not found in list.json
failed to get logo: unsuccessful status code: 404

❌ Asset 'umeevaloper1mkv4myppz2w0d3nr2dh9fv5kcwaaqnzfvnm6qf' (umee) not found in list.json

@cryptomanz1 cryptomanz1 merged commit b84514d into trustwallet:master Sep 5, 2023
julianariel pushed a commit to block-wallet/assets that referenced this pull request Oct 4, 2023
@sreblock @ElonXAIDogeMessi69PepeInu
@cryptomanz1 @SergeyMitk @ArtemGoryunov @erience @Cryptocool1 @trumpmagadev @covain @Legend-I @mwsoofy @curiouschonk
[Automated] Update from upstream repository [2023-09-05T12:00:02Z] (#221
26f0a53 
)

* Info for token ElonXAIDogeMessi69PepeInu (trustwallet#26233)

* Migration for SquidGrow & QANX (trustwallet#26248)

* Auto fix all (sanity and consistency)

* Create logo.png

* Create info.json

* Create logo.png

* Create info.json

* Delete logo.png

* Update info.json

* Delete logo.png

* Update info.json

* Update tokenlist.json

---------

Co-authored-by: cryptomanz1 <[email protected]>

* Rebranding Genesis Lab validator to Stakeflow (trustwallet#26118)

* Rebranding Genesis Lab to Stakeflow

* Solana logo fix.

* Logo size fixed to 256x256

* The logo.png file back.

* Delete logo.png

---------

Co-authored-by: Nazar <[email protected]>

* Add trending tokens (trustwallet#26263)

* Auto fix all (sanity and consistency)

* Create info.json

* Create logo.png

* Create info.json

* Create logo.png

* Create info.json

* Create logo.png

* Create info.json

* Create logo.png

---------

Co-authored-by: cryptomanz1 <[email protected]>

* binance/cardano/kava/nativeinjective/solana validator list updated (trustwallet#26311)

* near validator (trustwallet#26313)

* Fix list NEAR validators (trustwallet#26316)

* Delete logo.png

* Update list.json

* Delete logo.png

* Add Capitalrock (BEP20) (trustwallet#26306)

* Info for token Capitalrock

* Update info.json

---------

Co-authored-by: CryptoA <[email protected]>

* Info for token TRUMP (trustwallet#26302)

* Add 1inch promo image (trustwallet#26403)

* Add trending tokens (trustwallet#26456)

* Create info.json

* Create logo.png

* Create info.json

* Create logo.png

* Create info.json

* Create logo.png

* Create logo.png

* Create info.json

* Add Legend VI Validator Logo. (trustwallet#26457)

Co-authored-by: Legend-I <[email protected]>
Co-authored-by: Nazar <[email protected]>

* Info for token WEB3MET (trustwallet#26462)

Co-authored-by: CryptoA <[email protected]>

* Update info.json (trustwallet#26499)

* Add Graph Token (PoS) (trustwallet#26500)

0x5fe2B58c013d7601147DcdD68C143A77499f5531

https://polygonscan.com/address/0x5fe2b58c013d7601147dcdd68c143a77499f5531#code

https://polygonscan.com/token/0x5fe2b58c013d7601147dcdd68c143a77499f5531

---------

Co-authored-by: ElonXAIDogeMessi69PepeInu $BITCOIN <[email protected]>
Co-authored-by: Nazar <[email protected]>
Co-authored-by: cryptomanz1 <[email protected]>
Co-authored-by: SergeyMitk <[email protected]>
Co-authored-by: Artem <[email protected]>
Co-authored-by: Erience Solutions <[email protected]>
Co-authored-by: CryptoA <[email protected]>
Co-authored-by: TRUMP MAGA <[email protected]>
Co-authored-by: Ivan <[email protected]>
Co-authored-by: Legend-I <[email protected]>
Co-authored-by: Legend-I <[email protected]>
Co-authored-by: Mohammed Soofy Waseem <[email protected]>
Co-authored-by: curiouschonk <[email protected]>
@SergeyMitk
Copy link
Contributor Author

Hello, @cryptomanz1!
Our team has discovered that you made a mistake when you merged our pull-request #26118
In this commit you made changes to Band, Near and Umee that we did not originally make edcb2dd
And afterwards you just removed our Near validator from the list due to wrong id here d86832e
Could you please return our Near validator with the correct id: stakeflow.poolv1.near, and also change Umee id in the file https://github.com/trustwallet/assets/blob/master/blockchains/umee/validators/list.json from bandvaloper17s35yz3rglx7pnjppm4u9l9ht2k9mr9tq0eyh6 to umeevaloper1mkv4myppz2w0d3nr2dh9fv5kcwaaqnzfvnm6qf.

@cryptomanz1
Copy link
Contributor

cryptomanz1 commented Oct 20, 2023 •
edited
Loading

Hello!
fixed in this PR: #26616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@merge-fee-bot merge-fee-bot[bot] merge-fee-bot[bot] approved these changes

Assignees

@Cryptocool1 Cryptocool1

@cryptomanz1 cryptomanz1

Labels
Payment Status: Paid Fee completely paid for PR Payment Status: Requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SergeyMitk @cryptomanz1 @Cryptocool1