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

chore: bump registry to v11.3.1 #60

Merged
merged 1 commit into from
Sep 23, 2024
Merged

chore: bump registry to v11.3.1 #60

merged 1 commit into from
Sep 23, 2024

Conversation

conorsch
Copy link
Contributor

Closes #57 (again).

@conorsch conorsch requested a review from cronokirby September 20, 2024 18:45
@conorsch
Copy link
Contributor Author

Deployed from this branch to testnet infra, and confirmed that it resolves the error about registry not being found. That's good. However, I'm not convinced that the underlying asset info is correct, given the display:

recent-trades

Still, this patch fixes a crash loop on the current head of main for the testnet instance, so I suggest we merge it and continue to debug asset displays across the various frontends.

Copy link
Contributor

@cronokirby cronokirby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, agree that we can figure out potential issues and parsing stuff etc. later

@conorsch
Copy link
Contributor Author

Rebased to include changes from #62.

@conorsch conorsch merged commit 93e0e53 into main Sep 23, 2024
2 checks passed
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.

Update asset registry for penumbra-testnet-phobos-2
2 participants