denoland / denoland/deploy_feedback

New UI feedback

Open
#199 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
79
Forks
5
PR merge metrics
No merged PRs in 30d

Description

Overall this is a big improvement and I'm liking the simplicity of the new UI. Here are some thoughts and feedback on the new UI.

__Overview tab__
* "Recent Deployments" suggests to me they are old. Equally, deployments from 3 months ago doesn't necessarily feel 'recent' either. Possibly rename to "Non-production deployments"?
* Not enough space on desktop for commit message, which also spills into the commit time frame (e.g. "3 months ago"). You could possibly look to have a wider container on desktop as there's lot of unused space.
* I'm unclear what the green checkmark on the left of the deployment denotes
* It would be nice to have the committer name as alt text on the committer avatar (which would also make this more accessible)
* Summary box - I'm unclear if these stats are for all deployments or production only. Probably doesn't need a UI change, but something to consider for the docs or help somewhere.

__Analytics tab__
* Possibly add limits to this page. E.g. "Request Count" might show "1,245 of 3 million" or "1,245 of unlimited", etc.
* Is the CPU time accurate? Surprised to see my 3 requests consuming 320ms CPU time which would theoretically breaks the allowed limits?

__Logs tab__
* Took me awhile to understand what the `!` SVG icon meant on the left. Consider adding title/desc tags to the SVG graphics to make them more accessible and clear
* Make clear if you are looking at production logs, if you are actually looking at production logs
* I have a series of GET requests in the logs which returned HTTP 200, but have a red 'Crash' icon in the row. I have no idea what that means (doesn't help that it was from Feb so no idea what was going on then!).
* Log filtering would be nice

__Deployments tab__
* As per Overview page feedback, I'm not clear on the green checkmark and IMG/SVG accessibility would be nice.
* Unless I'm missing something, production deployment doesn't appear on this page? I think it should be on this page (it is a deployment after all) and clearly marked as production.
* Ideally search would include commit author too

__Settings tab__
* In Environment Variables, hyperlink Deno.env API to either https://doc.deno.land/deno/stable/~/Deno.env or https://deno.com/deploy/docs/runtime-api#deno-apis
* Possibly link to the Environments section of the docs (https://deno.com/deploy/docs/projects#environment-variables).
* It was unexpected to see the environment value disappear after entering it, though I now appreciate the purpose
* Not all projects have a web accessible endpoint on the root of the domain. Consider adding a setting for default view subroot. E.g. if I have a default sub-root of "/healthcheck", then clicking on "View" or any deployment link will add "/healthcheck" to the deployment URL to visit something I can actually look at.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.