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.

Sign up for GitHub

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

Jump to bottom

fix: rewards-related flicker #337

Closed
wants to merge 70 commits into from
Closed

fix: rewards-related flicker #337

wants to merge 70 commits into from

Conversation

truemiller
Copy link
Collaborator

@truemiller truemiller commented Sep 9, 2024 •
edited
Loading

follow-up to #330 for #306 #336

ensures rewards are loaded before displaying balances / earned rewards tag

solarw and others added 30 commits August 6, 2024 18:13
@solarw
initial win support
8967e9b
@solarw
Merge branch 'main' of github.com:valory-xyz/olas-operate-app into fe…
da1579a 
…at/win-support
@solarw
Merge branch 'staging' of github.com:valory-xyz/olas-operate-app into…
18c234b 
… feat/win-support
@solarw
win: process kill fixes
d1015e7
@solarw
Merge branch 'staging' of github.com:valory-xyz/olas-operate-app into…
0e5dfbf 
… feat/win-support
@solarw
win: tendermint installation automated
0930ac2
@solarw
helthchecker env vars to switch it off. small refactoring
e42ba6e
@solarw
Merge branch 'feat/contract-migration-ui' of github.com:valory-xyz/ol…
f793a03 
…as-operate-app into feat/win-support
@solarw
service hash id updated
bccc559
@solarw
use bundled in tendermint.exe, not download it
8f6cea6
@solarw
Merge branch 'staging' of github.com:valory-xyz/olas-operate-app into…
d06eb95 
… feat/win-support
@solarw
Merge branch 'staging' of github.com:valory-xyz/olas-operate-app into…
bc234f8 
… feat/win-support
@solarw
Merge branch 'staging' of github.com:valory-xyz/olas-operate-app into…
5d6b007 
… feat/win-support
@solarw
Merge branch 'main' of github.com:valory-xyz/olas-operate-app into fe…
b398318 
…at/win-support
@solarw
lint fixes
d80ef14
@jmoreira-valory
feat: add staking deployment vars
0adeefa
@truemiller
feat: added pearl beta 2 address & section
ef1bbe0
@jmoreira-valory
chore: add pearl beta 2 address
32872ad
@jmoreira-valory
Merge remote-tracking branch 'origin/feat/update_staking' into feat/u…
4f90bd5 
…pdate_staking
@mohandast52
chore: init
e5e107c
@truemiller
chore: comment env example
0ae8d9f
@truemiller
fix: isNil check for low balance
b86056a
@truemiller
fix: serviceFundRequirements coalesce
440f5e5
@truemiller
refactor: order and map staking contracts
b1da55f
@truemiller
fix: required olas when both staked and not staked
e9a53bb
@truemiller
feat: update StakingContractSection to support multi-contract
5d2d0fd
@truemiller
feat: updated STAKING_PROGRAM_META with deprecated and supported migr…
9f7e41b 
…ation targets
@truemiller
fix: isAddress checks to protect against 0xm
66eac9c
@truemiller
feat: added pearl_beta_2 to StakingContractInfo record
3291615
@truemiller
feat: add pearl_beta_2 Beta2 enum
93a96cb
mohandast52 and others added 25 commits September 4, 2024 14:49
@mohandast52
Merge pull request #315 from valory-xyz/mohan/staking-contract-ui
94df634 
feat: move the staking contract info to the main screen
@truemiller
fix: default contract fallback, instead of "not staked"
4de52d2
@truemiller
fix: broken import (kebabCase import)
d4578c6
@truemiller
Merge pull request #316 from valory-xyz/fix/selected-fallback
a192da8 
fix: change "Not Staked" to defaultStakingContract's name
@truemiller
bump: rc125
de4b0d9
@solarw
win build poetry version bump
d83f04c
@truemiller
fix: added minimum staking duration to migration condition
29d9edf
@solarw
fix for windows tendermint bundle in
4a0d4d7
@truemiller
Merge pull request #320 from valory-xyz/poetry-win-test
5745a56 
Poetry win test
@truemiller
Merge pull request #319 from valory-xyz/fix/hide-funding-until-comple…
0bb1fc9 
…ted-staking-period

fix: hide funding until completed staking period
@truemiller
fix: typo
e04d470
@solarw
fix for tendermint PATH env vars
fd212a7
@mohandast52
feat: Disable the staking contract button if the balance is low. (#317)
a707ddb 
* feat: disable staking contract button if low balance

* feat: update staking contract button popover message

* feat: add useNeedsFunds hook

* feat: integrate useNeedsFunds hook for disable staking contract

* update logic
@truemiller
refactor: remove OLAS requirement from notification
49ad26a
@truemiller
fix: commented link to broken site
9ffafde
@truemiller
Merge pull request #326 from valory-xyz/poetry-win-test
a4ae3ad 
fix: tendermint PATH environment variables
@mohandast52
feat: Staking info per contract (#325)
1b235d1 
* refractor: tiny format document

* feat: add InfoBreakdown component

* feat: replace OlasBalanceSection with BalanceBreakdown

* refactor: Update InfoBreakdown component and add InfoBreakdownList

* feat: add more contract methods for staking info

* move from different function inside the main one (Autonolas.ts)

* feat: add types

* refactor: Update StakingContractSection component and add StakingContractInfo

* other changes

* feat: minor

* feat: address Josh review changes
@mohandast52
feat: Add balance polling pause functionality to StakingContractSecti…
59bca90 
…on (#330)
@truemiller
Merge pull request #328 from valory-xyz/fix/remove-pandora-link
eed4cae 
fix: removed link to erroring pandora.computer
@truemiller
Merge pull request #327 from valory-xyz/fix/hardcoded-olas-running-no…
4336aad 
…tification

fix: remove hardcoded OLAS requirement from "now running" notification
@truemiller
bump: rc126
fd18e36
@truemiller
hotfix: undefined fixes in useStakingContractInfo hook for build 126
c8bd590
@truemiller
fix: ensure rewards are loaded
8684332
@truemiller
chore: TODO comments to address (DX related)
14f86a9
@truemiller
feat: added balance skeleton class (stops resizing when skeleton is h…
a25e05e 
…idden)
@truemiller truemiller changed the base branch from main to staging September 9, 2024 15:29
Base automatically changed from staging to main September 12, 2024 11:51
@truemiller truemiller changed the base branch from main to staging September 12, 2024 17:32
@truemiller truemiller closed this Sep 18, 2024
@truemiller
Copy link
Collaborator Author

closed as falled out of sync with staging, lots of conflicts
somewhat simple fix if reimplemented:
add isLoading state and polling pause to RewardsProvider, export from useRewardsHook, use in skeleton when awaiting balance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@truemiller @solarw @jmoreira-valory @mohandast52