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.

Sign up for GitHub

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

Jump to bottom

UI: actor overview #123

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

UI: actor overview #123

wants to merge 7 commits into from

Conversation

magik6k
Copy link
Collaborator

@magik6k magik6k commented Jul 26, 2024

This PR:

  • Adds a link to a new actor page in the actor overview table
      • a button to copy the actor address
  • Starts creating a new page with actor details
    • Miner actor balances
    • Miner Owner/Worker/Control address balances / states
      • Can show pending messages, recently sent messages
    • Chart of expirations, like in the actor overview but bigger
    • (Maybe later) Charts of power over time (+maybe projected with expirations)
    • (Maybe later) Chart of reward vesting schedule
    • Would be nice - projections of expected block wins and rewards
    • Links to block explorers, maybe

TODO:

  • Finish the frontend
  • Implement the backend

2024-07-26-121805_1341x587_scrot
2024-07-26-121831_1720x1187_scrot

@magik6k magik6k changed the title Feat/actor overview UI: actor overview Jul 26, 2024
@snadrus
Copy link
Contributor

snadrus commented Jul 29, 2024

how about landing this as-is (minus place-holders) and then we can add those on the next PR?

magik6k
magik6k commented Aug 13, 2024
View reviewed changes
web/static/actor/index.html Outdated Show resolved Hide resolved
web/static/actor/index.html Show resolved Hide resolved
web/static/actor/index.html Outdated Show resolved Hide resolved
@snadrus snadrus marked this pull request as ready for review August 14, 2024 22:27
@snadrus snadrus requested a review from LexLuthr August 14, 2024 22:28
LexLuthr
LexLuthr reviewed Aug 15, 2024
View reviewed changes
Copy link
Contributor

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add screenshots of the new page along with error handling screenshots in case method results in an error.

documentation/en/installation.md
@@ -163,7 +163,7 @@ git checkout <release version>
3. By default, a ‘multicore-sdr’ option is used in the proofs library. This feature is also used in FFI unless explicitly disabled. To disable building with the ‘multicore-sdr’ dependency, set `FFI_USE_MULTICORE_SDR` to `0`:\


```shell
```shell
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will misalign the docs.

Screenshot 2024-08-15 at 12 48 41 PM

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

@snadrus snadrus snadrus left review comments

@LexLuthr LexLuthr LexLuthr left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@magik6k @snadrus @LexLuthr