Skip to content

feat(graph-gateway): added indexers lookup table to graph network#371

Merged
LNSD merged 1 commit intomainfrom
feat-graph-gateway-topology-indexers-lut
Aug 11, 2023
Merged

feat(graph-gateway): added indexers lookup table to graph network#371
LNSD merged 1 commit intomainfrom
feat-graph-gateway-topology-indexers-lut

Conversation

@LNSD
Copy link
Contributor

@LNSD LNSD commented Aug 11, 2023

I decided to chunk into smaller PRs #364 since the conflicts with the main branch are making me rework certain parts.

This PR adds the following increments:

  • Added an Address<->Indexer lookup table to the GraphNetwork struct.
  • As all Indexer's URLs are guaranteed to be valid (see: 7f2f89aa-24c9-460b-ab1e-fc94697c4f4), use the indexer associated statu_url and cost_url methods.

@LNSD LNSD requested a review from Theodus August 11, 2023 13:02
@LNSD LNSD self-assigned this Aug 11, 2023
@LNSD LNSD force-pushed the feat-graph-gateway-topology-indexers-lut branch from a1cf174 to 726d349 Compare August 11, 2023 18:38
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

LGTM. Just some comments about the GUID links

@LNSD LNSD force-pushed the feat-graph-gateway-topology-indexers-lut branch from 726d349 to dbbb27f Compare August 11, 2023 18:42
@LNSD LNSD force-pushed the feat-graph-gateway-topology-indexers-lut branch from dbbb27f to 63ed029 Compare August 11, 2023 18:44
@LNSD LNSD changed the title fear(graph-gateway): added indexers lookup table to graph network feat(graph-gateway): added indexers lookup table to graph network Aug 11, 2023
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

👍

@LNSD LNSD merged commit 9fcf45e into main Aug 11, 2023
@LNSD LNSD deleted the feat-graph-gateway-topology-indexers-lut branch August 11, 2023 18:51
@Theodus Theodus mentioned this pull request Aug 21, 2023
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