-
Notifications
You must be signed in to change notification settings - Fork 56
Conversation
783f516
to
2011f51
Compare
2011f51
to
c9abbfe
Compare
c9abbfe
to
b164676
Compare
87da00b
to
48e7cd7
Compare
restake: { | ||
backgroundColor: "#EEFDFE", | ||
}, | ||
"validator-reward": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why you don't use camelCase here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea. I changed that to camelCase.
|
||
return ( | ||
<> | ||
{actions.map((subaction, subindex) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about
{actions.map((subaction, subindex) => { | |
{actions.map((subAction, subIndex) => { |
Way more readable I guess.
</TableElement> | ||
<DateTableElement> | ||
{subindex === 0 | ||
? moment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we don’t need moment
here as we switched to another library
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's what I found as well after rebase :)
48e7cd7
to
71ad7f0
Compare
Bundle size diff
|
71ad7f0
to
6defd21
Compare
This is a version of #937 with more context added to each record.
Preview links:
https://frontend-pr-1006-3r1e.onrender.com/beta/accounts/multisender.app.near
https://frontend-pr-1006-3r1e.onrender.com/beta/accounts/qbit.poolv1.near
https://frontend-pr-1006-3r1e.onrender.com/beta/accounts/9755042d33e148f87265c3343b69ed902939f166.lockup.near
https://frontend-pr-1006-3r1e.onrender.com/beta/accounts/foundation.near
https://frontend-pr-1006-3r1e.onrender.com/beta/accounts/82f09db4a072c66f89e8e226234181bb53f5ec24.lockup.near