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

Rewards update #1192

Merged
merged 36 commits into from
Nov 26, 2024
Merged

Rewards update #1192

merged 36 commits into from
Nov 26, 2024

Conversation

yuli-ferna
Copy link
Contributor

@yuli-ferna yuli-ferna commented Nov 25, 2024 •
edited
Loading

Copy of: #1187

image

HnryHan added 30 commits November 20, 2024 11:16
@HnryHan
Update text
be9db71 
- Optimism to Solana
- wstETH to USDs
- aWSTETH to Kamino USDs
@HnryHan
Change browser title from OP Rewards Dashboard to USDs Rewards Dashboard
088f2be
@HnryHan
Change OP units to USDs
07914ed
@HnryHan
Remove eWSETH field
0acd5ad
@HnryHan
Update overview api. TODO: Replace estimated rewards with proper fiel…
be484e1 
…d when implemented
@HnryHan
Update usersummary api
5f586d5
@HnryHan
Replace web3modal with reown appkit
01f04a0
@HnryHan
Remove @web3inbox
4845566
@HnryHan
Remove @reown/appkit-adapter-wagmi
e83dc25
@HnryHan
Remove web3modal
0926443
@HnryHan
Remove ensName truncate
396db26
@HnryHan
Remove viem and wagmi
e53d7b8
@HnryHan
@TanStack react query update
de614b4
@HnryHan
Text updates
3ae197e
@HnryHan
Use net_usds_supply_in_kamino for kamino usds balance
a6f1976
@HnryHan
Use bridged_usds instead of effective_bridged_usds
12e89e2
@HnryHan
Change balance to usd effective_usds_balance instead of usds_balance
5a1542b
@HnryHan
Change Total Projected Rewards to Weekly Rewards Pool
bea1702
@HnryHan
Add Est hourly rewads
2e6a937
@HnryHan
Update text and link to weekly rewards
c09581c
@HnryHan
Change to kamino held
cabd857
@HnryHan
Change to usds bridged
fa05c09
@HnryHan
USDSBalance
5ee11fe
@HnryHan
Fix Header text and link
df5075c
@HnryHan
Remove old subtext
dcd86f0
@HnryHan
Update formatNumber to use commas
4bb1dd3
@HnryHan
Remove trailing zeroes
aa8699d
@HnryHan
Fix formatInteger
784a8b6
@HnryHan
formatInteger, always show to 2 decimal places
f9efb4d
@HnryHan
Remove logs
c610144
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:07 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 10:14 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 10:15 — with GitHub Actions Inactive
Alex99y
Alex99y approved these changes Nov 26, 2024
View reviewed changes
Copy link
Contributor

@Alex99y Alex99y left a comment

Choose a reason for hiding this comment

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

I see that the subdomain changes instead of the routes, making it weird to define in the URL env part of the URL instead of the complete one. We should create a ticket to track this issue and attack it in the future in a correct way.

Approved.

@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:25 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Testnet November 26, 2024 13:32 — with GitHub Actions Inactive
@yuli-ferna yuli-ferna temporarily deployed to Cloudflare-Preview November 26, 2024 13:33 — with GitHub Actions Inactive
sebastianscatularo
sebastianscatularo approved these changes Nov 26, 2024
View reviewed changes
Copy link
Contributor

@sebastianscatularo sebastianscatularo left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@yuli-ferna yuli-ferna merged commit 6a6acb7 into main Nov 26, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alex99y Alex99y Alex99y approved these changes

@sebastianscatularo sebastianscatularo sebastianscatularo approved these changes

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
@yuli-ferna @sebastianscatularo @Alex99y @HnryHan