-
Notifications
You must be signed in to change notification settings - Fork 19
Blocks auto-update? #441
Comments
@sunnygleason do we have this kind of api? |
@manuel-calavera I was thinking of this as a front-end implementation:
Would you like to take a pass at implementing this? I'm happy to -- I just |
(moving to blocked since this is pending feedback) |
@sunnygleason i will check on it in coming days. thx! |
@sunnygleason i suggest to change design. instead of pulling txs on ui part and comparing them with already stored in store i suggest to have an additional api call to return count of new transaction happened after provided tx id. basically ui triggers api by timeout with providing most recent tx id and gets counter of new transactions happened after most recent in ui store. lmk what you think. |
@manuel-calavera that makes sense - so maybe modify like this?
Split out 2 responsibilities: refreshView(), and refreshInfo()
If there's a better name(s) for "info row" and "current view", I'm |
@sunnygleason sounds good, we need similar component for blocks table. i think we may need to show it only on 1st page of table. actually i am still not sure if we want pages vs load more on scroll. design mockups do not show next-prev pages selectors. you can make an api as i proposed or api + components if you want. I am ok to make components after you finish api. up to you. |
Our designs for transactions include a row to provide auto-update functionality:
I think we should implement this on blocks as well
The text was updated successfully, but these errors were encountered: