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

Basin UI (3) - Yield info + misc fixes #713

Merged

Conversation

Space-Bean
Copy link
Contributor

@Space-Bean Space-Bean commented Dec 10, 2023

In This PR:

  • Replaced Well Trading Fees column for Yield & added tooltip on hover
  • Updated tooltips throughout the app
  • modified Reserves component formatting
  • added well reserve tooltip displaying well reserves & TWAReserves if the well has MultiFlowPump
  • Disallow user from inputting an amount greater than their balance in 'Remove Liquidity'
  • Fixed a bug where duplicated data would cause the chart to display nothing
  • other minor housekeeping / formatting / typing along the way
  • Fix charts Y data points overflowing chart heights

To deploy an empty well for testing, you can run npx ts-node projects/examples/src/wells/deployNewWell.ts from the project root folder

@Space-Bean Space-Bean marked this pull request as ready for review December 10, 2023 20:36
const getSeedsWithWell = useCallback(
(well: Well | undefined) => {
if (!well?.lpToken) return undefined;
return sdk.tokens.findByAddress(well.lpToken.address)?.getSeeds();
Copy link
Contributor

@0xalecks 0xalecks Dec 13, 2023

Choose a reason for hiding this comment

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

In order for .getSeeds() to work, you MUST call sdk.refresh()

Note, this requires BIP39, which hasn't been deployed yet so this means we cannot merge this PR until after.

Happy to chat about this if you have questions. You don't need to set the seed for each token like I did in the UI PR below the line commented, that was only to update the UI's tokens... the SDK will update its tokens on refresh().

@Space-Bean Space-Bean merged commit a49fb92 into spacebean/dex-ui/design-fixes Dec 13, 2023
2 checks passed
@Space-Bean Space-Bean deleted the spacebean/dex-ui/yield-and-housekeeping branch December 13, 2023 19:58
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