Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: removed sequencer rewards command #1069

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

keruch
Copy link

@keruch keruch commented Oct 10, 2024

PR Standards

ADR link https://www.notion.so/dymension/ADR-Whitelisted-Relayer-Fee-Exemption-in-Rollapp-104a4a51f86a80e8ad94c916c87deb17
Closes dymensionxyz/dymension#1226


For Author:

  • Targeted PR against correct branch
  • Included the correct type prefix in the PR title
  • Linked to Github issue with discussion and accepted design
  • Targets only one github issue
  • Wrote unit and integration tests
  • All CI checks have passed
  • Added relevant godoc comments

For Reviewer:

  • Confirmed the correct type prefix in the PR title
  • Reviewers assigned
  • Confirmed all author checklist items have been addressed

After reviewer approval:

  • In case PR targets main branch, PR should be squashed and merged.
  • In case PR targets a release branch, PR should be rebased.

@keruch keruch self-assigned this Oct 10, 2024
@keruch keruch requested a review from a team as a code owner October 10, 2024 16:12
@keruch
Copy link
Author

keruch commented Oct 10, 2024

@artemijspavlovs MsgCreateSequencer and related messages are to be removed in the RDK repo, so i remove this command as well. Omri asked not to merge it until we release the version, so the PR will be dangling for a while.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sequencer zero-fee relayer address registration
1 participant