# Lighthouse UI Prototype Review

**OUTDATED, see revision 2: https://hackmd.io/BdJsIOICRhGmvRywGa4SWA**

This document is a review by Paul Hauner (and whoever else wants to contribute) on the [Lighthouse UI Prototype](https://www.figma.com/proto/eUmpbpqKK7VtqV0RYWlFAx/LH---Prototype?page-id=0%3A1&node-id=8%3A69594&viewport=1033%2C657%2C0.3116210103034973&scaling=contain) delivered by Aqeel on 2021/07/09.

## Missing Pages

- ~~Full validator onboarding flow.~~ Demonstrated via existing HTML code.
- Validator top-up
- Validator exit
- Recover Validator
- Import Validator

## Per-Page: Comments/Questions/Requests

### Page: Header (common)

- We want to ensure the user thinks it's OK if they don't always have MetaMask connected. It's only needed for deposit/top-up.
- Is the "Actvity" section of "Slasher" defined? I can't think of anything we need to put in there presently. IMO we leave this detail for future-works.

### Page: Footer (common)

- It might be nice to add a Github and Docs link next to the Discord one.

### Page: Dashboard

- Question: what is behind the `...` to the right of LH versions?
- Question: what is the icon on the right of the validators with 3 dots in a triangle?
- Maybe it links to validator management?
- Note: we will need to clearly define what the values for the "Validators" table will be.
- Note: we'll have to clearly define the possible "Alerts".
- Nit: Lighthouse versions present in lowercase `v1.4.0-dfc55fa`.
- Request: In "Logs":
- Rename "Warnings" -> "Errors".
- Rename "Errors" -> "Criticals".
- Rename "Logs Per Minute" to "Warns".

### Page: Analytics

I understand this page isn't designed and won't be included in the final designs. Fine by me :)

### Page: Validator Management

- Note: As per the dashboard, we'll need to clearly define the values in the "Validators" table (e.g., "Proposal", "Attestation", etc.)
- I'm not sure how to express the "Avg. Effectiveness" here.

### Page: Validator Management > Add Validator

Looks good!

### Page: Validator Management > Add Validator > Create New Validator

Looks good, but I think it's just a rough outline. I guess the flow is the same as onboarding.

### Page: Logs

Looks good!

### Page: Notifications

- Question: is this the same content as the "Alerts" panel on the dashboard?

## Global: Open Questions (for Sigma Prime)

- Define how to access ETH/USD, etc prices.
- How do we deal with the pricing information when we're on a testnet?
- The UI will need to know the name of the host for which it should read metrics. E.g., if Prom/Graf have mutliple nodes we need to only source one.
- How are we going to deal with validator naming? Store it in the VC?

## Global: Lighthouse Engineering Tasks

- How do we compute hourly/daily/weekly/monthly/total rewards?
- Ensure uptime is exposed for VC/BN.
- Expose the sync-status of the ETH1 node via the BN API.
- How to export logs to the UI?
- Support topup-deposits via the VC API.
- Support export via the VC API.
- Support backup via the VC API.
- Support exiting via the VC API.