fix: messed up api proto-gen

[aaronc]

Description

seems like I didn't get the pulsar alpha6 protogen right in #10669

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[codecov]

Codecov Report

Merging #10902 (e8a8fda) into master (0cb7fd0) will decrease coverage by 0.19%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master   #10902      +/-   ##
==========================================
- Coverage   64.87%   64.68%   -0.20%     
==========================================
  Files         628      610      -18     
  Lines       59689    58774     -915     
==========================================
- Hits        38722    38016     -706     
+ Misses      18770    18619     -151     
+ Partials     2197     2139      -58     

Impacted Files	Coverage Δ
x/auth/middleware/fee.go	80.00% <50.00%> (-0.91%)	⬇️
store/types/gas.go	84.44% <100.00%> (ø)
x/auth/middleware/middleware.go	73.21% <100.00%> (+0.99%)	⬆️
orm/encoding/ormkv/index_key.go
orm/encoding/ormkv/seq.go
orm/encoding/ormfield/timestamp.go
orm/encoding/ormkv/primary_key.go
orm/encoding/ormfield/int64.go
orm/encoding/ormkv/key_codec.go
... and 13 more

[tac0turtle]

lgtm, do you have to bump the version in the go.mod

[fdymylja]

But why are all the gRPC interfaces getting removed?

[aaronc]

But why are all the gRPC interfaces getting removed?

I have no idea. So is pulsar supposed to generate gRPC interfaces? I thought we had to use grpc-go

[atheeshp]

Is this related to fix the make proto-gen issue?

[fdymylja]

But why are all the gRPC interfaces getting removed?

I have no idea. So is pulsar supposed to generate gRPC interfaces? I thought we had to use grpc-go

pulsar should not generate gprc - only proto messages.

we need to use grpc-go to generate grpc stuff. it should even be in a different file.

we maybe need to add the grpc plugin to buf?

for instance, we need to generate gprc stuff to allow clients to query things in the chain.

[aaronc]

But why are all the gRPC interfaces getting removed?

I have no idea. So is pulsar supposed to generate gRPC interfaces? I thought we had to use grpc-go

pulsar should not generate gprc - only proto messages.

we need to use grpc-go to generate grpc stuff. it should even be in a different file.

we maybe need to add the grpc plugin to buf?

for instance, we need to generate gprc stuff to allow clients to query things in the chain.

That's what I thought. I can add that to this PR

[aaronc]

Is this related to fix the make proto-gen issue?

which make proto-gen issue?

[aaronc]

Just added grpc codegen in api/

[fdymylja]

Not tested, but let's merge it for now.

Will be experimenting to see if all works and will create issues accordingly!

Thank you @aaronc

[aaronc]

Not tested, but let's merge it for now.

Will be experimenting to see if all works and will create issues accordingly!

Thank you @aaronc

Thanks @fdymylja 👍