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

Doc: example app - client UI #221

Open
taqtiqa-mark opened this issue May 26, 2022 · 3 comments
Open

Doc: example app - client UI #221

taqtiqa-mark opened this issue May 26, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@taqtiqa-mark
Copy link

taqtiqa-mark commented May 26, 2022 •
edited
Loading

Thanks for the work on eventually. I'm trying to come to grips with the moving parts and encountered some trouble in getting a client to test the server setup I thought the following might be a useful addition to the documentation (README.md?)

To setup a Client

  1. Go to 'examples/bank-accounting'
  2. Execute cargo run
  3. Start another shell, execute podman run -it --rm --network="host" -v $(pwd)/proto:/protos fullstorydev/grpcui -plaintext localhost:10437
  4. See the gRPCui web page at http://0.0.0.0:8080/
  5. How to test the functionality via this client....
@taqtiqa-mark taqtiqa-mark added the bug Something isn't working label May 26, 2022
@taqtiqa-mark
Copy link
Author

taqtiqa-mark commented May 28, 2022 •
edited
Loading

Actually this is an error in starting gRPCui, but I wonder if this setup description is suitable as a Documentation PR to help fellow noobs?

The correct setup is:

  1. Start another shell, execute podman run -it --rm --network="host" -v $(pwd)/proto:/protos fullstorydev/grpcui -plaintext localhost:10437

@taqtiqa-mark taqtiqa-mark changed the title Example app: unspecific protocol error detected Doc: example app - client UI May 28, 2022
@ar3s3ru
Copy link
Collaborator

ar3s3ru commented Jun 2, 2022

Hey @taqtiqa-mark, thank you for reporting this 🙇🏻‍♂️

I think more documentation is always better. We can definitely add more information in a relevant README.md file 👍🏻
I cannot spot any difference from the two commands you posted in your comments. Is it the same?

@taqtiqa-mark
Copy link
Author

-insecure vs -plaintext

taqtiqa-mark added a commit to taqtiqa-mark/eventually-rs that referenced this issue Jun 11, 2022
@taqtiqa-mark
Add[Doc]: Client setup for bank accounting example
fd008b8 
Close issue get-eventually#221
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@taqtiqa-mark @ar3s3ru