-
Notifications
You must be signed in to change notification settings - Fork 139
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
Transactions Module #274
Comments
@jcx120 |
For |
hello @jcx120, in the historical TPS of the transactions module we should always bring the last 30 days fixed, or should we also have the possibility of filters such as 1 hour, 1 day, 30 days? |
@raphaelmartins1993 If this is easy to do, then let's have the 1hr, 1day, 30 day filter to keep it consistent with other metrics. Otherwise, let's keep it fixed at 30 days. |
Hi @jcx120 and @quietbits, we just did the transactions component with "Transactions Per Second" and "Historical TPS", what do you think of the layout? |
I think it looks fine. We can have UX @cbshin-stellar |
ok, we have a preview here: https://dashboard-pr293.prototypes.kube001.services.stellar-ops.com/?network=mainnet |
@jcx120 and @quietbits, another thing, I noticed that the "RECENT OPERATIONS: LIVE NETWORK" table on dashboard v1 some data is treated that is not returning in the horizon API like:
like others fields. Are these data retrieved at some point in an operation or can we disregard them? |
Cool! I'm assuming the Fees module and Ledger module are coming soon correct? |
yes, we are intending to finish this in the next days |
UX and visuals look great! no feedback to add, it's pretty 1:1 with the design so I'm happy with how the staging looks currently |
For Recent Operations, we can limit the number of items displayed to 10 rows. |
Hi @jcx120, we still need to check this account? I was chatting with @quietbits and I had this doubt? |
@lccoronel let's remove it, doesn't look critical to me |
@lccoronel For the "Transactions Per Second" Module, lets:
This is so the most important info is available at the top. thanks |
@lccoronel Looking good and thanks for the updates. One minor text change: |
Build a "Transactions" container component with the following elements:
Considerations:
The text was updated successfully, but these errors were encountered: