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

feat: initial implementation of grpc #14

Merged
merged 19 commits into from
Sep 21, 2023
Merged

Conversation

D4ryl00
Copy link
Contributor

@D4ryl00 D4ryl00 commented Sep 1, 2023

Add rule Makefile to build the protobuf: make api.generate

@D4ryl00 D4ryl00 self-assigned this Sep 1, 2023
@D4ryl00 D4ryl00 marked this pull request as draft September 1, 2023 08:41

// WithDefaultRemote init a default remote node address.
var WithDefaultRemote GnomobileOption = func(s *gnomobileService) error {
s.remote = "testnet.gno.berty.io:26657"
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the difference between this default remote and the one in the Client struct?
https://github.com/gnolang/gnomobile/blob/a07f32725fef3ec12c256445659d00afb81ade2e/service/client.go#L36

remote string
chainID string

activeAccount keys.Info
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does the active account need to be maintained in this struct? It seems that this duplicates the functionality of nameOrBech32 in the Client struct.
https://github.com/gnolang/gnomobile/blob/a07f32725fef3ec12c256445659d00afb81ade2e/service/client.go#L23

D4ryl00 and others added 15 commits September 15, 2023 11:05
@D4ryl00 D4ryl00 marked this pull request as ready for review September 21, 2023 07:58
@D4ryl00 D4ryl00 requested a review from jefft0 September 21, 2023 07:58
Copy link
Contributor

@jefft0 jefft0 left a comment

Choose a reason for hiding this comment

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

Works for me on iOS simulator.

@jefft0 jefft0 merged commit 415cb20 into gnolang:main Sep 21, 2023
@D4ryl00 D4ryl00 deleted the feat/add-grpc branch September 21, 2023 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants