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

Remove maps from other chains #711

Closed
karynemayer opened this issue Jun 6, 2023 · 2 comments · Fixed by #747
Closed

Remove maps from other chains #711

karynemayer opened this issue Jun 6, 2023 · 2 comments · Fixed by #747
Assignees
Labels
✨ Enhancement Improvements on existing features

Comments

@karynemayer
Copy link
Contributor

karynemayer commented Jun 6, 2023

Overview

We should remove the map because its hardcoded for telos network, showing only the block producers for Telos. Also, other chains don't necessarily want it a map.

Acceptance Criteria

  • Include a configuration to a chain to define if the map should appear or not
  • Set the configuration to false for every chain except telos and telos testnet.
@karynemayer karynemayer added the ✨ Enhancement Improvements on existing features label Jun 6, 2023
@eosiodetroit-adam
Copy link
Collaborator

@karynemayer So this would be an item that needs to be made configurable?

@eosiodetroit-adam eosiodetroit-adam added the 🔬 Needs Grooming Further information is required label Aug 2, 2023
@karynemayer karynemayer removed the 🔬 Needs Grooming Further information is required label Aug 14, 2023
@jaegerfe
Copy link
Collaborator

@karynemayer So this would be an item that needs to be made configurable?

We already have the flag for map display on the chain config. On #747 I turned off the map for the other networks.

@jaegerfe jaegerfe linked a pull request Sep 19, 2023 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Enhancement Improvements on existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants