-
Notifications
You must be signed in to change notification settings - Fork 191
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
feat: wire-up rfox staking #6925
Merged
Merged
Changes from 23 commits
Commits
Show all changes
94 commits
Select commit
Hold shift + click to select a range
2b9e72b
staking components
reallybeard 991181e
add staking components
reallybeard bba059a
fix translation
reallybeard b8b7948
move files around
reallybeard 8a0db0f
Merge branch 'develop' into WEB-6738
reallybeard 2e85097
Revert "move files around"
reallybeard 8a47fd3
revert
reallybeard 7b825ea
Revert "revert"
reallybeard babcc15
idfk
reallybeard 8b733f4
rename folders
reallybeard 4589be6
remove the older folders
reallybeard 167e701
Update StakeInput.tsx
reallybeard ada2d7d
translations
reallybeard c9d200a
use already defined translations
reallybeard 7b15060
update others
reallybeard fc18587
Update AddressSelection.tsx
reallybeard 169573c
[skip ci] wip: wire-up rfox staking
gomesalexandre 59e4ed7
feat: don't display gas row when approval required
gomesalexandre 8e0d3d4
[skip ci] feat: more todo
gomesalexandre 7cd8674
[skip ci] fix: derp
gomesalexandre fbb0bb1
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre 78237fa
[skip ci] fix: forgotten mergefix
gomesalexandre a68bcd9
[skip ci] fix: and another one
gomesalexandre 28d0514
feat: abi types inference in wagmi
gomesalexandre c6fc998
Merge branch 'develop' into feat_wire_up_rfox_staking_1
gomesalexandre 3a1be8d
feat: disable when no runeAddress
gomesalexandre b8a8c6c
feat: add thorchain validateAddress
gomesalexandre 35ce474
feat: wire-up validation
gomesalexandre 2058255
feat: approve
gomesalexandre 34565f5
feat: actual approve
gomesalexandre c8c5ca8
fix: i'm derp
gomesalexandre 40a89ec
feat: she works
gomesalexandre ac243f4
fix: types
gomesalexandre 1f35e70
feat: use constant
gomesalexandre db6e89f
feat: more cleanup
gomesalexandre 1875020
feat: tackle many TODOs
gomesalexandre 0fc7343
fix: generatedAssetData.json workaround
gomesalexandre bb89458
feat: regen asset data
gomesalexandre 31eda2e
feat: implement form validation in TradeAmountInput
gomesalexandre 6505601
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre 6c3b12e
feat: regen assetData
gomesalexandre 2b659a8
feat: fox on arb market-data monkey patch
gomesalexandre 4f8ba1c
feat: monkey-patch for now vs. regenerating all
gomesalexandre d955784
feat: nullish access of control when in the context of an undefined form
gomesalexandre 42662ae
fix: shit
gomesalexandre aa2f3a9
feat: input display either approval or actual stake fees
gomesalexandre 7adc540
feat: improve loading stateroos
gomesalexandre 3c49953
feat: don't fire queries when amount invalid
gomesalexandre 77471f5
feat: improve loading states some more
gomesalexandre 46be5df
feat: share of pool percentage
gomesalexandre 8f3d830
feat: cleanup useless mutation
gomesalexandre 18d3e3b
feat: actually revert mutation to benefit from loading states
gomesalexandre 78d3407
feat: leverage form some more
gomesalexandre d9a7583
feat: bump viem and wagmi to latest
gomesalexandre 7f85132
feat: move stakeFees to getFees
gomesalexandre 1621fff
feat: move getFees into react-queries
gomesalexandre d4c5400
fix: wallet doesn't serialize in JSON derp
gomesalexandre ec216b2
fix: types
gomesalexandre 1b430e8
feat: pass validation down
gomesalexandre b4e9ef5
feat: last cleanup
gomesalexandre b9a9776
feat: share of pool at confirm step
gomesalexandre 4cfe379
feat: cleanup TODO
gomesalexandre 11d5951
feat: remove comment
gomesalexandre 40b5b57
fix: only show recent transactions from connected wallet (#6938)
kaladinlight e914b43
chore: use effective coalesing for selector cache keys and download b…
woodenfurniture c23bc60
feat: form-based fee balance validation
gomesalexandre adc9912
feat: invalidate queries on from change
gomesalexandre ad1cf5f
fix: user share of pool
gomesalexandre 42e372c
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre 22ad71b
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre ca7ee8e
Merge branch 'develop' into feat_wire_up_rfox_staking_1
gomesalexandre 98b7203
fix: user share of pool
gomesalexandre acce670
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre 794a515
feat: rm todo
gomesalexandre e1eb627
feat: cherry-pick renaming bits from unstaking PR
gomesalexandre 1a1c9de
feat: cherry-pick cleanup bits from unstaking PR
gomesalexandre bb94069
fix: types
gomesalexandre 4aaee89
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre df3531f
fix: types after mergefix
gomesalexandre 492c65f
feat: fallback controller to unrug change address current view-layer
gomesalexandre b5b8d3d
fix: hydrate FOX market data in StakingInput
gomesalexandre cdf3dfe
feat: pass cryptoAmount in too in TradeAmountInput
gomesalexandre 86611b4
fix: tradeamountinput add missing prefix for crypto as opposite curre…
gomesalexandre 8196a79
fix: missing interpolation in translation
gomesalexandre bbacc30
fix: use stakingInfo vs. balanceOf for user balanceOf
gomesalexandre df5d6f2
feat: rm debug logs
gomesalexandre 4f537d5
feat: add toast
gomesalexandre 1e14154
feat: add isTransitioning
gomesalexandre 402a38d
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre 46e22b5
feat: add base to wagmi-config
gomesalexandre 8bd523b
feat: clearer naming
gomesalexandre 4e5a3f6
feat: conventional event-handler naming
gomesalexandre 1e92047
feat: omit undefined param in sendStakeTx and rename to handleStake
gomesalexandre 1123e2f
Merge remote-tracking branch 'origin/develop' into feat_wire_up_rfox_…
gomesalexandre File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Yeah we're bringing this back, no reason to reinvent the wheel when reading data in components - wagmi is just a wrapper around react-query and we would write ours, not so well optimized anyway, using viem.
Since wagmi wraps react-query and uses our viem client under the hood, that's literally the same we would write, but better. Nothing crazy huge either: https://bundlephobia.com/package/[email protected]
Note: this is the last v1 before wagmi published v2 which has viem v2 as a dependency, meaning we'd also have to bump viem to v2. Initially went with that and this is nothing super bad to tackle, but this PR is probably not the place for it so staying at v1 for now.
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.
Actually bumped viem and wagmi to latest, tackled all type errors and renames following breaking change. Now bundle is actually twice smaller 🎉