Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

chore: add Autonolas (OLAS) to validated-tokens #833

Closed
wants to merge 7 commits into from

Conversation

DavidMinarsch
Copy link

@DavidMinarsch DavidMinarsch commented Dec 28, 2023

Validate OLAS

Attestations (Please provide links):

Validation (Please check off boxes):

  • The metadata provided in the PR matches what is on-chain (Mandatory)
  • Does not duplicate the symbol of another token on Jupiter's strict list (If not, review will be delayed)
  • Is Listed on Coingecko / CMC (Optional, but helpful for reviewers)

Acknowledgement (Please check off boxes)

  • My change matches the format in the file (no spaces between fields).
  • My token is already live and trading on Jupiter.
  • !!! I read the README section on Community-Driven Validation and understand this PR will be only be reviewed when there is community support on Twitter.
  • Please make sure your pull request title has your token name. If it just says "Main", or "Validate", it will automatically be closed. PRs containing broken attestation or solscan links will also be closed.

validated-tokens.csv Outdated Show resolved Hide resolved
@kupermind
Copy link

kupermind commented Dec 28, 2023

[x] Mint address: Ez3nzG9ofodYCvEmw73XhQ87LWNYVRM2s7diB5tBZPyM
[x] Symbol: OLAS
[x] Token Decimals: 8
[x] Token icon: https://olas.network/images/olas-token-logo.png
[x] On-chain metadata matches
[x] No duplication on Jupiter's strict list
[x] Community tweet: https://x.com/kupermind/status/1740302083432333649?s=20

@oaksprout
Copy link

benlvb
benlvb previously approved these changes Dec 29, 2023
@Timoon21
Copy link
Contributor

Hey @DavidMinarsch I don't see any logoURI on the token metadata https://solscan.io/token/Ez3nzG9ofodYCvEmw73XhQ87LWNYVRM2s7diB5tBZPyM#metadata
Can you update your token metadata to match the logoURI in the PR?

@DavidMinarsch
Copy link
Author

Hey @DavidMinarsch I don't see any logoURI on the token metadata https://solscan.io/token/Ez3nzG9ofodYCvEmw73XhQ87LWNYVRM2s7diB5tBZPyM#metadata Can you update your token metadata to match the logoURI in the PR?

This is a bridged token. We have no control over the permissionless creation process

@DavidMinarsch
Copy link
Author

Hey @Timoon21 any thoughts on the response above? Would be great to get this merged.

@benlvb
Copy link
Collaborator

benlvb commented Jan 23, 2024

Screenshot 2024-01-23 at 19 06 44

There seems to be a discrepancy on the token name and ticker.

@DavidMinarsch
Copy link
Author

Screenshot 2024-01-23 at 19 06 44

There seems to be a discrepancy on the token name and ticker.

By this logic every token needs to be rejected. The scanner displays the name and the ticker. E.g. take the first entry in the list (https://github.com/jup-ag/token-list/blob/main/validated-tokens.csv) and compare it against solscan https://solscan.io/token/CbNYA9n3927uXUukee2Hf4tm3xxkffJPPZvGazc2EAH1

@DavidMinarsch
Copy link
Author

As to why the "(Wormhole)" now has been dropped by the scanner, I have no idea. This was not the case when this PR was opened. Would be great to get this merged one way or the other.

@raccoons-bot
Copy link
Collaborator

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs - please feel free to re-open when your project has more traction.

@raccoons-bot
Copy link
Collaborator

This pull request has not seen any activity since it was marked stale.
Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants