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

Enable indexing networks #596

Merged
merged 6 commits into from
Aug 14, 2024
Merged

Enable indexing networks #596

merged 6 commits into from
Aug 14, 2024

Conversation

bogdanfazakas
Copy link
Member

@bogdanfazakas bogdanfazakas commented Aug 7, 2024 •
edited
Loading

Fixes #503 .

Changes proposed in this PR:

  • added new INDEXER_NETWORKS env var to enable indexing on the networks available in
  • updated status handler and added indexingNetworks in config
  • updated dashboard indexer section and added supported networks
  • small tweaks in dashboard, updated logo footer, and load avg display
Screenshot 2024-08-07 at 16 08 03 Screenshot 2024-08-07 at 19 18 25

@bogdanfazakas bogdanfazakas self-assigned this Aug 7, 2024
@bogdanfazakas bogdanfazakas marked this pull request as ready for review August 7, 2024 17:39
jamiehewitt15
jamiehewitt15 approved these changes Aug 12, 2024
View reviewed changes
Copy link
Member

@jamiehewitt15 jamiehewitt15 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 to me

alexcos20
alexcos20 requested changes Aug 13, 2024
View reviewed changes
Copy link
Member

@alexcos20 alexcos20 left a comment

Choose a reason for hiding this comment

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

Can you add pls INDEXER_NETWORKS description in readme?
Otherwise, lgtm

mariacarmina
mariacarmina approved these changes Aug 13, 2024
View reviewed changes
Copy link
Member

@mariacarmina mariacarmina left a comment

Choose a reason for hiding this comment

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

lgtm

@bogdanfazakas bogdanfazakas merged commit 2f6e729 into develop Aug 14, 2024
8 checks passed
@bogdanfazakas bogdanfazakas deleted the feature/add-indexing-networks branch August 14, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mariacarmina mariacarmina mariacarmina approved these changes

@jamiehewitt15 jamiehewitt15 jamiehewitt15 approved these changes

@alexcos20 alexcos20 alexcos20 approved these changes

@paulo-ocean paulo-ocean Awaiting requested review from paulo-ocean paulo-ocean is a code owner

Assignees

@bogdanfazakas bogdanfazakas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bogdanfazakas @alexcos20 @jamiehewitt15 @mariacarmina