-
Notifications
You must be signed in to change notification settings - Fork 94
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
Additional leaderboards for voting subperiod #1188
Conversation
* add text ellipsis to the dapp card * update * Revert "update" This reverts commit 29c7d3f. * fix the arrow style of the ads section * fix builders card layout * fix dapp page short description
* Don't batch a single call * Ubstoppable grants background * Min staking amount warning condition fix * Code review update * Category title padding fix
Visit the preview URL for this PR (updated for commit b86f7d1): https://astar-apps--pr1188-feat-staking-v3-lead-i6eti30s.web.app (expires Thu, 22 Feb 2024 08:22:29 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db |
Hi @bobo-k2 Thank you! wanted to approve straight away to reduce workload but... please check
|
src/staking-v3/hooks/useDapps.ts
Outdated
@@ -98,13 +100,22 @@ export function useDapps() { | |||
return dapps.find((d) => d.owner === ownerAddress && d.state === DappState.Registered); | |||
}; | |||
|
|||
const fetchDappStats = async () => { |
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.
const fetchDappStats = async () => { | |
const fetchDappStats = async (): Promise<void> => { |
Pull Request Summary
Added transactions and users leaderboards for voting subperiod
Check list