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.

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

Make labels on latency charts more clear #20

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

haaawk
Copy link
Contributor

@haaawk haaawk commented Apr 14, 2023

Many people I showed the application to got confused by the Regional results.

Most of them started to understand what's going on only after I explained that Regional is a case when the edge function always runs in Washington instead of the closest region to the client.

It seems that the labels on the chart could be updated to remove this confusion and give more value to people that don't have time to dive deep into the logic of the app.

Many people I showed the application to got confused
by the Regional results.

Most of them started to understand what's going on
only after I explained that Regional is a case when
the edge function always runs in Washington instead of
the closest region to the client.

It seems that the labels on the chart could be updated
to remove this confusion and give more value to people
that don't have time to dive deep into the logic of the
app.

Signed-off-by: Piotr Jastrzebski <[email protected]>
@vercel
Copy link

vercel bot commented Apr 14, 2023

@haaawk is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@haaawk
Copy link
Contributor Author

haaawk commented Apr 14, 2023

@leerob Would you be open to changing the names of the labels? I've shown this app to many people and most of them got really confused by the Regional label. It becomes much easier for them to understand what's going on once I explain that Regional is really just always running the Edge function in Washington no matter where the client is.

@leerob
Copy link
Contributor

leerob commented May 2, 2024

I wonder how we can make this more clear now with the addition of Node.js as well.

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