graphql-voyager icon indicating copy to clipboard operation
graphql-voyager copied to clipboard

Add option to enter endpoint URL (fix #116)

Open FluorescentHallucinogen opened this issue 4 years ago • 5 comments

This pull request adds new "URL" tab into "Change schema" dialog to enter endpoint URL.

Before:

before

After:

after

FluorescentHallucinogen avatar Jan 24 '20 13:01 FluorescentHallucinogen

@IvanGoncharov Thanks for your comments! They are all great points. But I don't plan to fix them as part of this pull request because they are out of scope.

I'm guided by Google's Engineering Practices.

TL;DR: The product/code should get better than before merging a change, even if it isn’t perfect.

Before merging my change, users couldn't enter GraphQL endpoint URLs at all. After merging, they could. So the product is in better state than before. The quality of my change is not lower than the quality of the code. So it doesn't degrade the overall code health.

Nevertheless, I would be happy to fix them (and some other UI/UX issues I found) in the next iteration as a separate pull request. :wink:

FluorescentHallucinogen avatar Jan 26 '20 14:01 FluorescentHallucinogen

Deployed a live demo: https://graphql-voyager.web.app.

FluorescentHallucinogen avatar Apr 05 '20 09:04 FluorescentHallucinogen

Updated live demo: https://graphql-voyager.web.app.

graphql-voyager-url-tab

FluorescentHallucinogen avatar Apr 18 '20 16:04 FluorescentHallucinogen

This is an excellent feature that helps navigate the graph especially when there are multiple services which are federated. We have a use case where we would like to specify URL and let the voyager render the graph. Appreciate if this can be merged.

ravisastryk avatar Apr 29 '21 01:04 ravisastryk

Hey @IvanGoncharov, what's the state of this feature request? If the MR is too old to be merged I'll be happy to open a new mr with those changes (and we can discuss which changes to implement). I think this feature is a great UX / quality of life improvement to the product and should be considered.

nohehf avatar Feb 22 '23 01:02 nohehf