Skip to content

feat(tangle-dapp): Implement LS pool modal: Update commission#2630

Merged
drewstone merged 19 commits intodevelopfrom
yuri/ls-pool-management-flow
Nov 7, 2024
Merged

feat(tangle-dapp): Implement LS pool modal: Update commission#2630
drewstone merged 19 commits intodevelopfrom
yuri/ls-pool-management-flow

Conversation

@yuri-xyz
Copy link
Copy Markdown
Contributor

@yuri-xyz yuri-xyz commented Oct 31, 2024

Summary of changes

Provide a detailed description of proposed changes.

  • ➕ Added the liquid staking pool management modal that allows setting & updating the commission rate of a specific liquid staking pool.
  • 🔧 Refactored some of the text used in the Claim Airdrop page to make it more natural sounding and explicit.
  • 🐛 Fixed bugs on the Claim Airdrop page related to the claiming input's address being wrongly capitalized ([BUG] Addresses on claim dropdown are not consistent #2598).
  • 🖌️ Updated & improved some font sizes and designs across the dApp for improved legibility ([TASK] Small UI font visibility & improvements #2601).

Proposed area of change

Put an x in the boxes that apply.

  • apps/tangle-dapp
  • apps/tangle-cloud
  • libs/webb-ui-components

Associated issue(s)

Specify any issues that can be closed from these changes (e.g. Closes #233).

Screen Recording

If possible provide screenshots and/or a screen recording of proposed change.

ls-update-commission-modal.mp4

@yuri-xyz yuri-xyz self-assigned this Oct 31, 2024
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 31, 2024

Deploy Preview for tangle-cloud ready!

Name Link
🔨 Latest commit 3dff04d
🔍 Latest deploy log https://app.netlify.com/sites/tangle-cloud/deploys/672bf06b3029ce0008faee42
😎 Deploy Preview https://deploy-preview-2630--tangle-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 31, 2024

Deploy Preview for tangle-dapp ready!

Name Link
🔨 Latest commit 3dff04d
🔍 Latest deploy log https://app.netlify.com/sites/tangle-dapp/deploys/672bf06be44ffe000858a6cf
😎 Deploy Preview https://deploy-preview-2630--tangle-dapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@drewstone
Copy link
Copy Markdown
Contributor

Don't forget to provide short video demos :)!

@yuri-xyz
Copy link
Copy Markdown
Contributor Author

yuri-xyz commented Nov 3, 2024

Don't forget to provide short video demos :)!

@drewstone Yes, on some I omit them if they're more 'technical' changes, but on new UI features like this I'll be sure to include them.

@yuri-xyz yuri-xyz changed the title feat(tangle-dapp): Implement LS pool management modals feat(tangle-dapp): Implement LS pool modal: Update commission Nov 4, 2024
Copy link
Copy Markdown
Contributor

@drewstone drewstone left a comment

Choose a reason for hiding this comment

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

Looks good, comments on style/comments and other misc things.

Comment thread apps/tangle-dapp/components/LiquidStaking/LsMyPoolsTable.tsx
Comment thread apps/tangle-dapp/components/LiquidStaking/LsMyPoolsTable.tsx
Comment thread apps/tangle-dapp/components/LiquidStaking/LsMyPoolsTable.tsx Outdated
Comment thread apps/tangle-dapp/components/LiquidStaking/LsMyPoolsTable.tsx
Comment thread apps/tangle-dapp/data/liquidStaking/tangle/useLsSetCommissionTx.ts Outdated
Comment thread apps/tangle-dapp/utils/isSubstrateAddress2.ts Outdated
@yuri-xyz yuri-xyz marked this pull request as ready for review November 6, 2024 22:35
@drewstone drewstone merged commit 17ef399 into develop Nov 7, 2024
@drewstone drewstone deleted the yuri/ls-pool-management-flow branch November 7, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants