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 Superfluid Token (SUP) #145

Closed
wants to merge 1 commit into from

Conversation

d10r
Copy link
Contributor

@d10r d10r commented Feb 14, 2025

No description provided.

@d10r
Copy link
Contributor Author

d10r commented Feb 14, 2025

from the CI run:

BRIDGE null
REMOTE_TOKEN null
BRIDGE null
REMOTE_TOKEN null
/home/runner/work/token-lists/token-lists/src/verify.ts:141
      throw new Error("Tokens do not point at each other");      

We implemented the L2 token following the instructions in https://docs.optimism.io/app-developers/bridging/standard-bridge
This refers to IOptimismMintableERC20, which the token implements.
This was successfully tested on testnet a few weeks ago.

The verifier seems to expect a different interface to be implemented.
Can somebody please clarify what's missing and where this is documented?

@d10r
Copy link
Contributor Author

d10r commented Feb 14, 2025

After some more investigation, I believe that the verification script is wrong and created an issue: #146

@AlexBHarley
Copy link
Member

Closed with #149

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

Successfully merging this pull request may close these issues.

2 participants