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.

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

Add new dashboard/assets #2607

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Add new dashboard/assets #2607

wants to merge 12 commits into from

Conversation

kattylucy
Copy link
Collaborator

Description

This pull request...

#

Approvals

  • Dev

Screenshots

Impact

@kattylucy kattylucy changed the base branch from main to 2595_dashboard_menu_setup January 27, 2025 16:14
Copy link

github-actions bot commented Jan 27, 2025

PR deployed in Google Cloud
URL: https://app-pr2607.k-f.dev
Commit #: d0dc5e0
To access the functions directly check the corresponding deploy Action

Copy link

github-actions bot commented Jan 27, 2025

PR deployed in Google Cloud
URL: https://pr2607-app-ff-production.k-f.dev
Commit #: d0dc5e0
To access the functions directly check the corresponding deploy Action

Base automatically changed from 2595_dashboard_menu_setup to main January 28, 2025 10:18
@kattylucy kattylucy force-pushed the 2585_assets_page branch 8 times, most recently from a6ef4d3 to 5e7a8c8 Compare January 30, 2025 10:58
@@ -157,13 +157,13 @@ export function AssetTransactions({ pool }: { pool: Pool }) {
})
}
}
}, [transactions, setCsvData])
}, [transactions, setCsvData, explorer])
Copy link
Collaborator

Choose a reason for hiding this comment

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

explorer shouldn't be here, because it's not stable. Would just eslint ignore instead

children: ReactNode
}

export const SelectedPoolsProvider: React.FC<SelectedPoolsProviderProps> = ({ children }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

React.FC is obsolete

return loans?.find((loan) => loan.id === assetId)
}

export function useAvailableFinancing(poolId: string, assetId: string) {
const isTinlakePool = poolId.startsWith('0x')
const isTinlakePool = poolId.length === 1 && poolId[0].startsWith('0x')
Copy link
Collaborator

Choose a reason for hiding this comment

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

const isTinlakePool = poolId.startsWith('0x') seems fine here, as poolId is not an array here

@kattylucy kattylucy force-pushed the 2585_assets_page branch 6 times, most recently from 541b922 to 915c940 Compare February 6, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants