feat(spaceward): fix governance types #761
Conversation
|
Hey @alex-nax and thank you for opening this pull request! 👋🏼 It looks like you forgot to add a changelog entry for your changes. Make sure to add a changelog entry in the 'CHANGELOG.md' file. |
WalkthroughWalkthroughThe changes involve modifications to governance-related files within the Cosmos SDK, including updates to hooks, utility functions, and protocol buffer definitions. Key alterations include renaming variables for better metadata handling, deprecating certain parameters, adding new fields for proposals, and enhancing documentation across multiple files. Additionally, new RPC methods and message types have been introduced to support governance operations, improving overall structure and clarity. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant GovernanceModule
participant ProposalService
Client->>GovernanceModule: Submit Proposal
GovernanceModule->>ProposalService: Process Proposal
ProposalService-->>GovernanceModule: Proposal ID
GovernanceModule-->>Client: Confirmation with Proposal ID
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
Early access features: disabledWe are currently testing the following features in early access:
Note:
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
There was a problem hiding this comment.
Actionable comments posted: 1
Outside diff range, codebase verification and nitpick comments (2)
spaceward/src/features/governance/util.ts (1)
24-25: Consider providing informative error messages for better debugging and troubleshooting.The changes remove the descriptive error information that was previously provided when parsing metadata fails. This may impact the clarity of error reporting and make it harder to identify the cause of the failure.
Consider restoring the previous behavior of providing informative error messages. For example:
-title: "", // "Unexpected metadata", -summary: "", // hasKey("message", e) ? e.message : "No description", +title: "Unexpected metadata", +summary: hasKey("message", e) ? e.message : "No description",Providing descriptive error messages can greatly aid in debugging and troubleshooting issues related to metadata parsing.
wardenjs/proto/cosmos/gov/v1/query.proto (1)
147-159: LGTM!The changes in the
QueryParamsResponsemessage type are a step towards a more streamlined approach in handling governance parameters. The deprecation of the individual parameter fields (voting_params,deposit_params,tally_params) and the introduction of the consolidatedparamsfield of typeParamssuggest that users should prefer using the newparamsfield for accessing governance parameters.This change improves the structure and usability of the governance parameters by consolidating them into a single field. It also aligns with the best practice of deprecating old fields and providing a new, more efficient alternative.
To further enhance the clarity and usability of the
QueryParamsResponsemessage type, consider adding a comment above theparamsfield to explicitly mention that it consolidates all the governance parameters and should be preferred over the deprecated fields.
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files ignored due to path filters (29)
wardenjs/src/codegen/cosmos/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/module/v1/module.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/genesis.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/gov.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/query.lcd.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/query.rpc.Query.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/query.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/tx.amino.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/tx.registry.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/tx.rpc.msg.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1/tx.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/genesis.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/gov.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/query.lcd.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/query.rpc.Query.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/query.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/tx.rpc.msg.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/cosmos/gov/v1beta1/tx.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/gogoproto/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/google/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/shield/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/slinky/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/tendermint/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/bundle.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/warden/v1beta3/events.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/warden/v1beta3/key.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/warden/v1beta3/keychain.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/warden/v1beta3/signature.tsis excluded by!wardenjs/src/codegen/**wardenjs/src/codegen/warden/warden/v1beta3/tx.tsis excluded by!wardenjs/src/codegen/**
Files selected for processing (11)
- spaceward/src/features/governance/hooks.ts (1 hunks)
- spaceward/src/features/governance/util.ts (1 hunks)
- wardenjs/proto/cosmos/gov/module/v1/module.proto (1 hunks)
- wardenjs/proto/cosmos/gov/v1/genesis.proto (1 hunks)
- wardenjs/proto/cosmos/gov/v1/gov.proto (3 hunks)
- wardenjs/proto/cosmos/gov/v1/query.proto (10 hunks)
- wardenjs/proto/cosmos/gov/v1/tx.proto (5 hunks)
- wardenjs/proto/cosmos/gov/v1beta1/genesis.proto (2 hunks)
- wardenjs/proto/cosmos/gov/v1beta1/gov.proto (6 hunks)
- wardenjs/proto/cosmos/gov/v1beta1/query.proto (10 hunks)
- wardenjs/proto/cosmos/gov/v1beta1/tx.proto (4 hunks)
Additional context used
buf
wardenjs/proto/cosmos/gov/module/v1/module.proto
5-5: import "cosmos/app/v1alpha1/module.proto": file does not exist
(COMPILE)
Additional comments not posted (70)
wardenjs/proto/cosmos/gov/module/v1/module.proto (1)
8-19: LGTM!The
Modulemessage definition looks good:
- Follows the protocol buffer style guide.
- Has clear comments explaining the purpose and default values of the fields.
- Correctly uses the
optiondeclaration to set the Go import path.The code changes are approved.
wardenjs/proto/cosmos/gov/v1beta1/genesis.proto (6)
16-17: LGTM!The change to include
(amino.dont_omitempty) = truefor thedepositsfield is approved. It ensures that empty repeated fields are included in the serialized output, which is consistent with the updates made to other similar fields.
19-20: LGTM!The change to include
(amino.dont_omitempty) = truefor thevotesfield is approved. It ensures that empty repeated fields are included in the serialized output, which is consistent with the updates made to other similar fields.
22-23: LGTM!The change to include
(amino.dont_omitempty) = truefor theproposalsfield is approved. It ensures that empty repeated fields are included in the serialized output, which is consistent with the updates made to other similar fields.
24-25: LGTM!The changes to the
deposit_paramsfield are approved:
- Including
(amino.dont_omitempty) = trueis consistent with the updates made to other fields.- Updating the comment improves clarity without changing the core functionality.
26-27: LGTM!The changes to the
voting_paramsfield are approved:
- Including
(amino.dont_omitempty) = trueis consistent with the updates made to other fields.- Updating the comment improves clarity without changing the core functionality.
28-29: LGTM!The changes to the
tally_paramsfield are approved:
- Including
(amino.dont_omitempty) = trueis consistent with the updates made to other fields.- Updating the comment improves clarity without changing the core functionality.
wardenjs/proto/cosmos/gov/v1/genesis.proto (5)
20-28: Deprecation changes look good.The deprecation of the
deposit_params,voting_params, andtally_paramsfields in favor of theparamsfield is approved. The deprecation messages provide clear guidance to use theparamsfield instead.
29-32: Addition of theparamsfield is approved.The new
paramsfield consolidates all the parameters related to the x/gov module, enhancing the organization and clarity of theGenesisStatemessage. The field is introduced with the appropriate version annotation.
33-39: Addition of theconstitutionfield is approved.The new
constitutionfield allows chain builders to articulate their vision and ideals at genesis. The field is well-documented, and its immutability is clearly stated. The introduction of the field in cosmos-sdk 0.50 is appropriately noted.
Line range hint
1-19:
Line range hint
40-41:spaceward/src/features/governance/hooks.ts (2)
47-47: LGTM!The code changes are approved. The aliasing of
summaryandtitleas_summaryand_titlerespectively, allows for a fallback mechanism in the returned object.
51-52: LGTM!The code changes are approved. The fallback mechanism for
descriptionandnameproperties enhances the robustness of the hook by ensuring that if the parsed metadata does not provide values, the original proposal values are still utilized.wardenjs/proto/cosmos/gov/v1beta1/tx.proto (6)
10-10: LGTM!The code changes are approved.
14-17: LGTM!The code changes are approved.
37-52: LGTM!The code changes are approved.
59-60: LGTM!The code changes are approved.
66-78: LGTM!The code changes are approved.
Also applies to: 89-101
112-130: LGTM!The code changes are approved.
wardenjs/proto/cosmos/gov/v1/query.proto (6)
15-17: LGTM!The
ConstitutionRPC method is a valuable addition to theQueryservice, enabling clients to query the chain's constitution. The method is correctly defined with its corresponding request and response message types.
61-61: LGTM!The
QueryConstitutionRequestmessage type is appropriately defined as the request type for theQuery/ConstitutionRPC method. The empty message type correctly indicates that no parameters are required for the request.
64-66: LGTM!The
QueryConstitutionResponsemessage type is properly defined as the response type for theQuery/ConstitutionRPC method. Theconstitutionfield of typestringis suitable for returning the chain's constitution in the response.
44-44: LGTM!The updated comment for the
DepositRPC method enhances clarity by accurately describing that the method queries single deposit information based onproposalIDanddepositAddr.
76-76: LGTM!The updated comment for the
proposalfield in theQueryProposalResponsemessage type enhances clarity by accurately describing that the field represents the requested governance proposal.
188-188: LGTM!The updated comment for the
depositsfield in theQueryDepositsResponsemessage type enhances clarity by accurately describing that the field represents the requested deposits.wardenjs/proto/cosmos/gov/v1beta1/query.proto (11)
9-9: LGTM!The addition of the Amino import is approved.
40-40: LGTM!The updated comment provides better clarity on the functionality of the
DepositRPC method.
64-64: Verify the impact on clients.The addition of the
(amino.dont_omitempty) = trueoption to theproposalfield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
88-89: LGTM!The updated comment provides better clarity on the purpose of the
proposalsfield. The addition of therepeatedkeyword to the field declaration is syntactically correct.
103-103: LGTM!The updated comment provides better clarity on the purpose of the
voterfield in theQueryVoteRequestmessage.
109-110: Verify the impact on clients.The updated comment provides better clarity on the purpose of the
votefield.However, the addition of the
(amino.dont_omitempty) = trueoption to thevotefield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
124-125: Verify the impact on clients.The updated comment provides better clarity on the purpose of the
votesfield.However, the addition of the
(amino.dont_omitempty) = trueoption to thevotesfield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
141-145: Verify the impact on clients.The addition of the
(amino.dont_omitempty) = trueoption to thevoting_params,deposit_params, andtally_paramsfields ensures that they are always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
163-163: Verify the impact on clients.The addition of the
(amino.dont_omitempty) = trueoption to thedepositfield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
177-178: Verify the impact on clients.The updated comment provides better clarity on the purpose of the
depositsfield.However, the addition of the
(amino.dont_omitempty) = trueoption to thedepositsfield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
193-193: Verify the impact on clients.The addition of the
(amino.dont_omitempty) = trueoption to thetallyfield ensures that it is always included in the serialized output, even if empty.Please verify that this change does not break any existing clients that rely on the previous serialization behavior where empty fields were omitted.
wardenjs/proto/cosmos/gov/v1/tx.proto (13)
11-12: LGTM!The added import statements are necessary and serve the following purposes:
amino/amino.proto: Provides support for Amino encoding.google/protobuf/timestamp.proto: Provides support for timestamp fields.
18-18: LGTM!The added service option
(cosmos.msg.v1.service) = truecorrectly indicates that theMsgservice is a Cosmos SDK message service.
36-40: LGTM!The
UpdateParamsRPC method is correctly added to theMsgservice and serves the purpose of updating the x/gov module parameters. The method signature and documentation are appropriate.
42-45: LGTM!The
CancelProposalRPC method is correctly added to theMsgservice and serves the purpose of canceling a governance proposal. The method signature and documentation are appropriate.
52-84: LGTM!The updates to the
MsgSubmitProposalmessage are correctly implemented and serve the following purposes:
- The
messagesfield allows for more complex interactions through the execution of arbitrary messages.- The
titleandsummaryfields provide more context for proposals.- The
expeditedfield allows for expedited proposals.- The Amino encoding options ensure backwards compatibility.
97-97: LGTM!The Amino encoding option
(amino.name) = "cosmos-sdk/v1/MsgExecLegacyContent"is correctly added to theMsgExecLegacyContentmessage and provides a descriptive name for the message, aiding in its identification and usage within the SDK.
111-123: LGTM!The updates to the
MsgVotemessage are correctly implemented and serve the following purposes:
- The Amino encoding option
(amino.name) = "cosmos-sdk/v1/MsgVote"provides a descriptive name for theMsgVotemessage, aiding in its identification and usage within the SDK.- The
metadatafield allows for attaching arbitrary metadata to the vote, providing more flexibility.
132-144: LGTM!The updates to the
MsgVoteWeightedmessage are correctly implemented and serve the following purposes:
- The Amino encoding option
(amino.name) = "cosmos-sdk/v1/MsgVoteWeighted"provides a descriptive name for theMsgVoteWeightedmessage, aiding in its identification and usage within the SDK.- The
metadatafield allows for attaching arbitrary metadata to the weighted vote, providing more flexibility.
153-162: LGTM!The updates to the
MsgDepositmessage are correctly implemented and serve the following purposes:
- The Amino encoding option
(amino.name) = "cosmos-sdk/v1/MsgDeposit"provides a descriptive name for theMsgDepositmessage, aiding in its identification and usage within the SDK.- The
cosmos.AddressStringscalar type for thedepositorfield ensures that the field contains a valid Cosmos address string.
171-182: LGTM!The
MsgUpdateParamsmessage is correctly implemented and serves the purpose of updating the x/gov module parameters. The message includes appropriate fields:
- The
authorityfield specifies the address that controls the module.- The
paramsfield specifies the parameters to update.The message documentation is also clear and indicates that it is available since cosmos-sdk 0.47.
184-188: LGTM!The
MsgUpdateParamsResponsemessage is correctly implemented and serves the purpose of defining the response structure for executing aMsgUpdateParamsmessage. The message documentation is clear and indicates that it is available since cosmos-sdk 0.47.
193-200: LGTM!The
MsgCancelProposalmessage is correctly implemented and serves the purpose of defining the request type for canceling a proposal. The message includes appropriate fields:
- The
proposal_idfield specifies the unique ID of the proposal.- The
proposerfield specifies the account address of the proposer.The message documentation is also clear and indicates that it is available since cosmos-sdk 0.50.
206-213: LGTM!The
MsgCancelProposalResponsemessage is correctly implemented and serves the purpose of defining the response structure for executing aMsgCancelProposalmessage. The message includes appropriate fields:
- The
proposal_idfield specifies the unique ID of the proposal.- The
canceled_timefield specifies the time when the proposal is canceled.- The
canceled_heightfield specifies the block height at which the proposal is canceled.The message documentation is also clear and indicates that it is available since cosmos-sdk 0.50.
wardenjs/proto/cosmos/gov/v1beta1/gov.proto (10)
10-10: LGTM!The import statement for
amino/amino.protois correct.
14-14: LGTM!The
(gogoproto.goproto_getters_all)option is set correctly tofalse.
36-36: LGTM!The comment accurately describes the
optionfield in theWeightedVoteOptionmessage.
39-40: LGTM!The comment accurately describes the
weightfield in theWeightedVoteOptionmessage.
42-43: LGTM!The
(gogoproto.customtype)and(amino.dont_omitempty)options for theweightfield are set correctly.
52-53: LGTM!The
(amino.name)option for theTextProposalmessage is set correctly tocosmos-sdk/TextProposal.
56-58: LGTM!The comment accurately describes the
titlefield in theTextProposalmessage.
69-80: LGTM!The comments accurately describe the
proposal_id,depositor, andamountfields in theDepositmessage. The(amino.dont_omitempty)option for theamountfield is set correctly.
87-122: LGTM!The comments accurately describe the various fields in the
Proposalmessage. The(amino.dont_omitempty)option is set correctly for the relevant fields.
Line range hint
129-152: LGTM!The comments accurately describe the enum values of the
ProposalStatusenum.wardenjs/proto/cosmos/gov/v1/gov.proto (10)
11-11: LGTM!The code change is approved.
31-35: LGTM!The code changes are approved.
41-48: LGTM!The code changes are approved.
53-110: LGTM!The code changes are approved.
136-142: LGTM!The code changes are approved.
149-161: LGTM!The code changes are approved.
167-201: LGTM!The code changes are approved.
203-270: LGTM!The code changes are approved.
87-110: Verify the usage of the new fields added to theProposalmessage in the codebase.Ensure that the new fields
title,summary,proposer,expedited, andfailed_reasonadded to theProposalmessage are being used correctly in other parts of the codebase.Run the following script to verify the usage of the new fields:
167-201: Verify the usage of the deprecated messages in the codebase.Ensure that the deprecated
DepositParams,VotingParams, andTallyParamsmessages are not being used in other parts of the codebase.Run the following script to verify the usage of the deprecated messages:
|
|
||
| package cosmos.gov.module.v1; | ||
|
|
||
| import "cosmos/app/v1alpha1/module.proto"; |
There was a problem hiding this comment.
Fix the missing import file.
The static analysis tool flags that the imported file cosmos/app/v1alpha1/module.proto does not exist. This will cause compilation errors.
Please ensure that the imported file exists at the specified path. If you need assistance with this, I can help you locate the correct file or open a GitHub issue to track this task. Let me know if you would like me to assist further.
Tools
buf
5-5: import "cosmos/app/v1alpha1/module.proto": file does not exist
(COMPILE)
…dd-genesis-space (#723) * validate bench32 in cmd add-genesis-keychain add-genesis-space * in executeAnalyzers return error on bad contract address (#716) * in executeAnalyzers return error on bad contract address * update changelog * validate bench32 in cmd add-genesis-keychain add-genesis-space * fix spelling and duplicates * * Added KeybaseId, Url and Name fields to keychain * Added changelog * Review comments * Code review fixes * Regenerated model --------- Co-authored-by: Archie <artur.abliazimov@gmail.com> * Fix of localnet.just (#726) * Fixed localnet.just * Fixed localnet.just * Remove the GMP default params from genesis (#727) * feat(faucet): new faucet (#479) * feat(faucet): new faucet This version of the Faucet does not have any limitations per user, only limitations are daily token supply * fix: minor css changes for mobile * fix(faucet): add daily refresh and token refresh job * fix(faucet): fix bugs and ci * fix(faucet): remove unnessary go.mod files, replace dockerfile faucet-v2 * chore(faucet): remove unnessary variables and modify types this will modify the types of DailyLimit and Batchinterval to remove unnessary conversions and remove Cooldown and BatchInterval * fix(faucet): acquire a lock when setting the new supply * fix(faucet): removed locking where it's not needed other enhancements as well, the loading and setting context to correct place * fix(faucet): converted background to much smaller image * feat(faucet): make the run out of daily nicer * feat(faucet): add new style * feat(faucet): add check that same address cannot be multiple time in one batch * feat(faucet): add metrics * fix(faucet): Dockerfile libwasmvm was missing access rights * feat(faucet): add pre input option to address form --------- Co-authored-by: Jon Heywood <94459819+jjheywood@users.noreply.github.com> * chore(wardend): bump ibc-go to v8.4.0 * docs: Update Keychain SDK (#706) * Upgrade keychain docs Update go version Update json for keychain-fees Update chain-id * Add categorization to Keychain SDK * Categorization * Add Keychain SDK + Description * Create a new tutorial Add basic info for keychain Implement main func with - handleKeyRequest and handleSignRequest Write test function * fixed a link in the Tools menu * quick fixes * quick fixes * quick fixes * remove ) --------- Co-authored-by: Margarita Skomorokh <ijonele@gmail.com> * fixed links to the keychain sdk (#733) * Made KeychainFees = nil invalid * ci: add check to ensure go.mod is tidy * chore: go mod tidy * fix(wardend): ICAHostKeeper initialization requires QueryRouter to be set Breaking change introduced in ibc-go v8.3 * feat(spaceward): key creation in sidebar (#728) * fix key icon in sidebar cleanup * fix getProider call * bump ethers version; fix erc20 send * build: use new justfile commands * fix(x/warden): make KeychainFees non-nullable * fix(x/warden): allow empty KeychainFees * cosmoshubtestnet hotfix; do not fail on wrong message type (#746) * feat(faucet): configurable chain name (#747) this change adds configurable chain/faucet name to the site * fix(faucet): chain variable missing from struct (#749) This PR adds the missing Chain variable that missed in #747 * #561 Keychain fees should not be paid in advance (#662) * Added keychain max wanted fees handling * Implemented new testcase & snapshot * Added annotations * Added keychain fee max value specification * Code review fixes * Changed validation messages * fix(x/act): add support for sdk.Coins in CLI flags * Changed type to sdk.Coin * Updated CHANGELOG.md * Implemented escrow accounts for keychain fees * Added CHANGELOG.md line * Regenerated keychain snapshot * Updated keychain deduction fee code * Added test case for fee deduction with --max-keychain-fee * Fixed actions auto parsing for cli * Fixed bug with not forking fees deduction and context missing values * Docker versions update * Update cmd/wardend/cmd/gen-keychains.go Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Proto rebuild * Merged changes from #560 * Code-review fixes * Implemented integration test for escrow account * Code review fixes * Made KeychainFees = nil invalid * Fixed error message * Keychain fee validations * Code review fixes * Merge conflict fixes * Lint fixes --------- Co-authored-by: Antonio Pitasi <antonio@pitasi.dev> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * feat(spaceward): update for rebrand (#748) * feat(spaceward): New theme added * fix(spaceward): Some theme fixes * fix(spaceward): Fixed keychains * fix(spaceward): Some theme fixes and metamask * fix(spaceward): Some more * fix(spaceward): Space Mono * fix(spaceward): fix for space mono font issue (#750) * fix(spaceward): updated welcome message (#751) * fix(faucet): fix daily limit i64 (#755) This fixes issue where user puts high uward amount to daily limit and go cannot parse it due to int restrictions * Add chain upgrade instructions for v0.4.2 (#757) * Add chain upgrade instructions for v0.4.2 * Update v0.4.2.md * update v0.4.2 Co-authored-by: Joonas Lehtimäki <joonas.lehtimaki@gmail.com> * remove reference to side-car --------- Co-authored-by: Joonas Lehtimäki <joonas.lehtimaki@gmail.com> * fixed spelling (#758) * fixed spelling * deleted the tip, fixed spelling * update slinky * fix(x/act): simplify prepareHandlerContext to always use sdk.Context * updated gov proto; fix governance types (#761) * Fix returned field (#765) * fix(spaceward): Fix logos (#767) * fix(spaceward): Fix logos * fix(spaceward): updated logo for WC connection and updated hover fill colour --------- Co-authored-by: Jon Heywood <94459819+jjheywood@users.noreply.github.com> * fix(faucet): tx page loading (#768) * feat(faucet): add new metric dailySupply * fix(faucet): page keeps loading even if tx was finished --------- Co-authored-by: backsapc <backsapce@hotmail.com> Co-authored-by: Archie <artur.abliazimov@gmail.com> Co-authored-by: Joonas Lehtimäki <joonas.lehtimaki@gmail.com> Co-authored-by: Jon Heywood <94459819+jjheywood@users.noreply.github.com> Co-authored-by: Antonio Pitasi <antonio@pitasi.dev> Co-authored-by: Aliasgar Merchant <44069404+alijnmerchant21@users.noreply.github.com> Co-authored-by: Margarita Skomorokh <ijonele@gmail.com> Co-authored-by: Margarita Skomorokh <margaret.skomorokh@anychart.com> Co-authored-by: Alex D <alex.d.nax@gmail.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Alice <37526212+laniakea42@users.noreply.github.com> Co-authored-by: Tyler <48813565+technicallyty@users.noreply.github.com>
updated gov proto; fix governance types
Summary by CodeRabbit
New Features
Constitutionfor querying the chain's constitution.UpdateParamsandCancelProposal.MsgSubmitProposalto include a field for submitting arbitrary messages.Improvements
paramsfield for better organization.Deprecations
GenesisStateand other messages as deprecated, encouraging the use of new structures.