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

[Gateway] Scaffold the unstake gateway message #67

Merged
merged 3 commits into from
Oct 16, 2023
Merged

Conversation

h5law
Copy link
Contributor

@h5law h5law commented Oct 14, 2023

Summary

Ran

ignite scaffold message unstake-gateway --module gateway --signer address --yes

Summary generated by Reviewpad on 16 Oct 23 18:06 UTC

This pull request includes various changes to multiple files:

  1. A new file named tx_unstake_gateway.go is added in the x/gateway/client/cli directory. It defines a command CmdUnstakeGateway for unstaking a gateway and includes transaction flags.

  2. In the x/gateway/types package, a new file named message_unstake_gateway.go is added. It defines the message type MsgUnstakeGateway for unstaking a gateway, along with necessary methods and helper functions.

  3. A new test file for the MsgUnstakeGateway type in the types package is added. It tests the ValidateBasic function for different scenarios.

  4. The file module_simulation.go includes changes related to simulating the MsgUnstakeGateway message.

  5. The file gateway/types/codec.go includes changes in the registration of MsgUnstakeGateway message type.

  6. The file tx.proto in the pocket/gateway directory includes changes related to adding a new RPC method for unstaking a gateway.

  7. The go.mod file includes changes in the dependencies of the project.

  8. A new file named msg_server_unstake_gateway.go in the x/gateway/keeper package includes the initial structure for handling unstaking of a gateway.

  9. In the x/gateway/simulation package, a new file named unstake_gateway.go includes the initial structure for simulating the unstaking of a gateway.

  10. The file tx.go includes changes related to the variable DefaultRelativePacketTimeoutTimestamp and the addition of the CmdUnstakeGateway() command.

  11. The file docs/static/openapi.yml includes changes in the definitions of MsgUnstakeGatewayResponse, MsgStakeGatewayResponse, and Params.

These changes add new functionality and improve the existing codebase. Let me know if you need further assistance!

Issue

Fixes (in part) 19

Type of change

Select one or more:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Testing

  • Run all unit tests: make test_all_unit
  • Verify Localnet manually: See the instructions [here](TODO: add link to instructions)

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have performed a self-review of my own code
  • I have commented my code, updated documentation and left TODOs throughout the codebase

@h5law h5law added the gateway Changes related to the Gateway actor label Oct 14, 2023
@h5law h5law added this to the Shannon TestNet milestone Oct 14, 2023
@h5law h5law requested a review from Olshansk October 14, 2023 19:22
@h5law h5law self-assigned this Oct 14, 2023
Olshansk
Olshansk previously approved these changes Oct 16, 2023
Base automatically changed from scaffold_gateway_stake to main October 16, 2023 18:00
@h5law h5law dismissed Olshansk’s stale review October 16, 2023 18:00

The base branch was changed.

@h5law h5law requested a review from Olshansk October 16, 2023 18:06
@h5law h5law merged commit b0678e0 into main Oct 16, 2023
3 checks passed
@h5law h5law deleted the scaffold_gateway_unstake branch October 16, 2023 18:36
bryanchriswhite added a commit that referenced this pull request Oct 18, 2023
* pokt/main:
  [Service] Added the `Service` type
  Updated the command for unit tests in the PR template
  Updated PR template
  [Miner] chore: scaffold create-claim message (#43)
  [E2E] Add basics of the E2E suite (#62)
  [Gateway] Scaffold the unstake gateway message and nothing else (#67)
  [Gateway] Scaffold the gateway stake message and nothing else (#66)
  Add placeholder for auto-generated packages
okdas pushed a commit that referenced this pull request Nov 14, 2024
Ran
```
ignite scaffold message unstake-gateway --module gateway --signer address --yes
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gateway Changes related to the Gateway actor
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants