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 to GRPC message types #160

Merged
merged 47 commits into from
Sep 16, 2024
Merged

change to GRPC message types #160

merged 47 commits into from
Sep 16, 2024

Conversation

kwtalley
Copy link
Collaborator

This PR moves to GRPC message types instead of Stargate. To reduce the package size and simplify encoding, serde has been removed.

You will need to update your contracts, especially if you have been using the types in provwasm_std as inputs/outputs to your contract.

closes #157

kwtalley added 21 commits August 12, 2024 14:15
@kwtalley
migrate to buf v2
bd79ac6
@kwtalley
remove old buf templates since only one is used now
ade3ed6
@kwtalley
add new jsonschema transformer to add to enum derives
f72cc9f
@kwtalley
comment out old serde derive transformers
736b68d
@kwtalley
remove custom buf template from generator projects
495a4ae
@kwtalley
update code generator to use common buf v2 template
e4abecd
@kwtalley
update main code gen project types
736db23
@kwtalley
add syn to proto-build cargo manifest
fec9b84
@kwtalley
try updating to prost 0.13
add409a
@kwtalley
change proc-macro queries to grpc
7097d41
@kwtalley
initial generated cosmos types without serde
828e94b
@kwtalley
initial generated capability types without serde
876d602
@kwtalley
initial generated cosmwasm types without serde
cac39ab
@kwtalley
initial generated tendermint types without serde
7377489
@kwtalley
initial generated ibc types without serde
c014cc1
@kwtalley
initial generated provenance types without serde
6cbaacd
@kwtalley
update marker contract for grpc work
fd2d983
@kwtalley
update name contract for grpc work
d6522b3
@kwtalley
comment out mod serde from provwasm-std lib
7ae1d0b
@kwtalley
add optimize goal to contracts makefile for simplicity
8ada75a
@kwtalley
fix grpc response decoding type. type is wrapped in ResponseQuery type
8d6e8ec
@kwtalley kwtalley added enhancement New feature or request smart contract Smart Contract implementations provwasm provwasm (cosmwasm for Provenance) labels Aug 30, 2024
@kwtalley kwtalley self-assigned this Aug 30, 2024
@kwtalley kwtalley requested a review from Taztingo as a code owner August 30, 2024 04:59
@kwtalley kwtalley marked this pull request as draft September 6, 2024 04:56
@kwtalley kwtalley marked this pull request as ready for review September 13, 2024 19:34
Taztingo
Taztingo approved these changes Sep 16, 2024
View reviewed changes
Copy link
Collaborator

@Taztingo Taztingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kwtalley kwtalley merged commit 0d5efd2 into main Sep 16, 2024
8 checks passed
@kwtalley kwtalley deleted the issue/157 branch September 16, 2024 17:37
@kwtalley kwtalley mentioned this pull request Sep 27, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Taztingo Taztingo Taztingo approved these changes

Assignees

@kwtalley kwtalley

Labels
enhancement New feature or request provwasm provwasm (cosmwasm for Provenance) smart contract Smart Contract implementations
Projects
Provenance Core Protocol Team
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate to GRPC
2 participants
@kwtalley @Taztingo