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

Replace time and size with URL in history dropdown (#677) #721

Merged
merged 1 commit into from Jan 24, 2018

Conversation

Projects
None yet
4 participants
@iansu
Contributor

iansu commented Jan 19, 2018

Closes #677

image

@welcome

This comment has been minimized.

welcome bot commented Jan 19, 2018

💖 Thanks for opening this pull request! 💖

To help make this a smooth process, please be sure you have first read the
contributing guidelines.

@gschier

Nice, this is awesome! Thanks @iansu

@gschier gschier merged commit 3ee969a into getinsomnia:develop Jan 24, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@welcome

This comment has been minimized.

welcome bot commented Jan 24, 2018

Congrats on merging your first pull request! 🎉🎉🎉 You're helping make Insomnia awesome! 🙌

@iansu

This comment has been minimized.

Contributor

iansu commented Jan 24, 2018

Thank you. I use Insomnia regularly so happy to contribute. 😀

@profnandaa

This comment has been minimized.

profnandaa commented Jan 24, 2018

Thank you for pulling this off @iansu 👏

@roboslone

This comment has been minimized.

roboslone commented Feb 7, 2018

This actually breaks my workflow, can this behaviour be optional?

@gschier

This comment has been minimized.

Collaborator

gschier commented Feb 8, 2018

Hey @roboslone. What's your workflow? I'd be open to making it configurable, but maybe there's a way to modify the new interface to satisfy your workflow.

@gschier

This comment has been minimized.

Collaborator

gschier commented Feb 8, 2018

Probably best to discuss this in a new issue so it doesn't get lost. Do you want to create one?

@roboslone

This comment has been minimized.

roboslone commented Feb 8, 2018

@gschier my workflow is that I use Insomnia to test backend handlers that I'm currently working on. I make changes on backend, send a request, then make some other changes. After several iterations I need response times to determine which changes made response time lower. Seeing response times in history is really useful in that case.

@gschier

This comment has been minimized.

Collaborator

gschier commented Feb 8, 2018

Thanks for the info. I actually just added those tags back so your workflow should be back to normal in the next release.

image

@roboslone

This comment has been minimized.

roboslone commented Feb 8, 2018

Thanks! Looking forward for an update.

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