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

Change validator type to sequencer type #77

Open
3 tasks
mtsitrin opened this issue Jan 30, 2023 · 2 comments
Open
3 tasks

Change validator type to sequencer type #77

mtsitrin opened this issue Jan 30, 2023 · 2 comments
Assignees
@mtsitrin
Labels
C:sequencers enhancement New feature or request good first issue Good for newcomers

Comments

@mtsitrin
Copy link
Collaborator

mtsitrin commented Jan 30, 2023 •
edited
Loading

Currently the sequencer type is the same as the validator in staking module.

It actually makes more sense to be similar to the sequencer struct in the hub

  • define Sequencer struct in proto
  • use Sequencer in all internal functions
  • Have Sequencer to Validator method, to support some Staking interface calls
@mtsitrin mtsitrin self-assigned this Jan 30, 2023
@mtsitrin mtsitrin added enhancement New feature or request C:sequencers labels Jan 30, 2023
@omritoptix omritoptix changed the title Refactor validator type to sequencer type Change validator type to sequencer type Sep 12, 2023
@mtsitrin mtsitrin added the good first issue Good for newcomers label Mar 14, 2024
@omritoptix
Copy link
Contributor

@mtsitrin can this be closed?

@mtsitrin
Copy link
Collaborator Author

mtsitrin commented May 7, 2024

no, we didn't implemented it yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mtsitrin mtsitrin

Labels
C:sequencers enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@omritoptix @mtsitrin