[Morse->Shannon Migration] scaffold: MsgClaimMorseSupplier
#1094
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit 71f5112)
…--signer authority
…nnon_dest_address morse_src_address morse_signature --response morse_src_address --response claimed_balance:coin --response claimed_at_height:int
…ctor/stake_supplier
…caffold/msg_claim_morse_supplier
…issues/1034/refactor/stake_app
…ctor/stake_supplier
…r' into issues/1034/scaffold/msg_claim_morse_supplier * pokt/issues/1034/refactor/stake_supplier: fix: linter error chore: remove redundant field & update field indices [Code Health] fix: support zero kvcache TTL (#1108) chore: review feedback improvements [Morse->Shannon Migration] chore: add gateway, app & supplier keeper deps (#1079)
Co-authored-by: Daniel Olshansky <[email protected]>
…issues/1034/refactor/stake_app
…ake_app * pokt/main: [Morse->Shannon Migration] scaffold `MsgClaimMorseApplication` message (#1080)
Co-authored-by: Daniel Olshansky <[email protected]>
…morse_app * pokt/main: [Morse->Shannon Migration] refactor: app staking logic as keeper methods (#1091)
…ctor/stake_supplier
…ake_supplier * pokt/main: [Morse->Shannon Migration] implement `MsgClaimMorseApplication` handler (#1082)
…caffold/msg_claim_morse_supplier
…g_claim_morse_supplier * pokt/main: [Morse->Shannon Migration] refactor: supplier `msgServer` methods to `Keeper` (#1093)
bryanchriswhite
changed the base branch from
issues/1034/refactor/stake_supplier
to
main
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
bryanchriswhite
added a commit
that referenced
this pull request
Mar 11, 2025
…morse_supplier * pokt/HEAD: [Morse->Shannon Migration] scaffold: `MsgClaimMorseSupplier` (#1094) # Conflicts: # api/poktroll/migration/tx.pulsar.go # proto/poktroll/migration/tx.proto # tests/integration/migration/morse_claim_supplier_test.go # x/migration/keeper/msg_server_claim_morse_supplier.go # x/migration/keeper/msg_server_claim_morse_supplier_test.go # x/migration/simulation/claim_morse_supplier.go # x/migration/types/message_claim_morse_supplier.go # x/migration/types/message_claim_morse_supplier_test.go # x/migration/types/tx.pb.go
bryanchriswhite
deleted the
issues/1034/scaffold/msg_claim_morse_supplier
branch
red-0ne
added a commit
that referenced
this pull request
Mar 12, 2025
## Summary ```bash ignite scaffold message claim_morse_supplier --module migration --signer shannon_dest_address morse_src_address morse_signature stake:coin service_config --response morse_src_address --response claimed_balance:coin --response service_id --response claimed_supplier_stake:coin --response claimed_at_height:int --response supplier ``` Changes: - Scaffold `MsgClaimMorseSupplier` - Add `TODO_UPNEXT` comments ## Issue - Issue: #1034 ## Type of change Select one or more from the following: - [x] New feature, functionality or library - [ ] Consensus breaking; add the `consensus-breaking` label if so. See #791 for details - [ ] Bug fix - [ ] Code health or cleanup - [ ] Documentation - [ ] Other (specify) ## Sanity Checklist - [x] I have updated the GitHub Issue `assignees`, `reviewers`, `labels`, `project`, `iteration` and `milestone` - [ ] For docs, I have run `make docusaurus_start` - [x] For code, I have run `make go_develop_and_test` and `make test_e2e` - [ ] For code, I have added the `devnet-test-e2e` label to run E2E tests in CI - [ ] For configurations, I have update the documentation - [x] I added TODOs where applicable --------- Co-authored-by: Daniel Olshansky <[email protected]> Co-authored-by: red-0ne <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Changes:
MsgClaimMorseSupplierTODO_UPNEXTcommentsIssue
Type of change
Select one or more from the following:
consensus-breakinglabel if so. See [Infra] Automatically add theconsensus-breakinglabel #791 for detailsSanity Checklist
assignees,reviewers,labels,project,iterationandmilestonemake docusaurus_startmake go_develop_and_testandmake test_e2edevnet-test-e2elabel to run E2E tests in CI