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

Token registry #87

Merged
merged 10 commits into from
Oct 13, 2023
Merged

Token registry #87

merged 10 commits into from
Oct 13, 2023

Conversation

anondev2323
Copy link
Contributor

@anondev2323 anondev2323 commented Oct 12, 2023 •
edited
Loading

  • Script checks for invalid values in config and throw error if found
  • Script compiles a list of updates
  • checkForeignAssetConfig checks for updates and logs the changes needed
  • updateForeignAssetConfig updates the tokens list
  • Add CI for determining if the list is out of date

TODO (gonna handle this later):

  • Check this list before fetching foreign assets/decimals

@anondev2323 anondev2323 marked this pull request as draft October 12, 2023 20:09
@anondev2323 anondev2323 marked this pull request as ready for review October 13, 2023 18:23
@anondev2323 anondev2323 changed the title WIP: Token registry Token registry Oct 13, 2023
barnjamin
barnjamin reviewed Oct 13, 2023
View reviewed changes
connect/src/wormhole.ts Outdated Show resolved Hide resolved
@barnjamin barnjamin merged commit 4ab499a into develop Oct 13, 2023
1 check failed
@barnjamin barnjamin deleted the token-registry branch October 19, 2023 19:58
barnjamin pushed a commit that referenced this pull request Oct 23, 2023
@anondev2323 @barnjamin
Token registry (#87)
5d58fbd
barnjamin pushed a commit that referenced this pull request Nov 25, 2023
@anondev2323 @barnjamin
Token registry (#87)
4b2cf3f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@barnjamin barnjamin barnjamin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anondev2323 @barnjamin