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

ci: add indexing node for dex-explorer #3825

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

conorsch
Copy link
Contributor

We're starting work on a new webapp that will provide a visual frontend for the DEX. That work is currently housed in [0]. Similar to cuiloa [1], we must enable CometBFT indexing to Postgres, so the application (yet to be written) can query those events. Strictly speaking, it's possible to reuse a db connection across multiple apps, particularly if we're careful about ensuring read-only access, but duplicating the node config per-app will simplify debugging in the near-term.

These changes have been manually applied for both environments. URLs may change in the future.

[0] https://github.com/penumbra-zone/dex-explorer
[1] https://github.com/penumbra-zone/cuiloa

We're starting work on a new webapp that will provide a visual frontend
for the DEX. That work is currently housed in [0]. Similar to cuiloa
[1], we must enable CometBFT indexing to Postgres, so the application
(yet to be written) can query those events. Strictly speaking, it's
possible to reuse a db connection across multiple apps, particularly if
we're careful about ensuring read-only access, but duplicating the node
config per-app will simplify debugging in the near-term.

These changes have been manually applied for both environments. URLs may
change in the future.

[0] https://github.com/penumbra-zone/dex-explorer
[1] https://github.com/penumbra-zone/cuiloa
@conorsch conorsch merged commit 11d3c7d into main Feb 15, 2024
6 checks passed
@conorsch conorsch deleted the dex-explorer-fullnode branch February 15, 2024 17:32
@cratelyn cratelyn added the A-dex Area: Relates to the dex label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dex Area: Relates to the dex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants