alexandrehtrb / Pororoca

An API testing tool with support for HTTP/2 and HTTP/3. Alternative to Postman.
https://pororoca.io
Other
500 stars 38 forks source link

feature: Show WebSocket connection response status code and headers #55

Closed tetropolix closed 11 months ago

tetropolix commented 11 months ago
alexandrehtrb commented 11 months ago

Hello @tetropolix , welcome!

tetropolix commented 11 months ago

I have updated the code based on suggestions

tetropolix commented 11 months ago

I have updated the code. I'm not sure if I should always rebase to one commit and then force push, or is it more conveniet (common practice) to gradually push commits and let the PR evolve.

alexandrehtrb commented 11 months ago

I have updated the code. I'm not sure if I should always rebase to one commit and then force push, or is it more conveniet (common practice) to gradually push commits and let the PR evolve.

Don't worry, you can use both ways, I think that gradually push is easier.

alexandrehtrb commented 11 months ago

@tetropolix , thanks for your contribution!

I will merge into develop.

There are some things about the UI that I don't know how to improve yet... my screen is small and the exchanged messages look squeezed... I'll see if I come up with an idea of how to fix it, maybe an alternative layout? And I'll mark you in the PR.

image

tetropolix commented 11 months ago

Thanks for your help and the opportunity for contributing