Skip to content

Update client app UX#141

Open
adamradocz wants to merge 4 commits intodavidfowl:mainfrom
adamradocz:UpdateClientAppUX
Open

Update client app UX#141
adamradocz wants to merge 4 commits intodavidfowl:mainfrom
adamradocz:UpdateClientAppUX

Conversation

@adamradocz
Copy link
Copy Markdown
Contributor

  • Fixed the typos in the Client App.
  • The while now re-render the console if the user goes back to the "main" menu.

@davidfowl
Copy link
Copy Markdown
Owner

Why?

@adamradocz
Copy link
Copy Markdown
Contributor Author

adamradocz commented Mar 15, 2022

I wanted to try a protocol multiple times, and it was annoying to restart the app for that. If you don't agree just delete the PR.

@davidfowl
Copy link
Copy Markdown
Owner

I had to look again because some of these options don't let quit, they run forever. This is only useful for rabbit mq right?

@adamradocz
Copy link
Copy Markdown
Contributor Author

adamradocz commented Mar 15, 2022

I specifically used it for the custom protocol, but the protocol had to be modified. That modification hasn't been included, because I created a PR (#130) for that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants