ENG-5747 feat(launchpad,graphql): get relic/protocol points on launchpad rewards page #1092
Conversation
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
apps/launchpad/app/.server/env.ts
Outdated
| @@ -67,6 +68,7 @@ export function getEnv() { | |||
| process.env.MULTIVAULT_ADDRESS_BASE_MAINNET, | |||
| ORIGIN_URL: process.env.ORIGIN_URL, | |||
| PRIVY_APP_ID: process.env.PRIVY_APP_ID, | |||
| HASURA_POINTS_ENDPOINT: process.env.HASURA_POINTS_ENDPOINT, | |||
There was a problem hiding this comment.
Do we also need this on the client?
| return useQuery({ | ||
| queryKey: ['get-relic-points', { address }], | ||
| queryFn: () => { | ||
| console.log('Query function executing with address:', address) |
There was a problem hiding this comment.
Logger instead or we can prob remove
| return { | ||
| address, | ||
| dehydratedState: dehydrate(queryClient), | ||
| initialParams: { |
There was a problem hiding this comment.
Did you have any trouble from omitting address from the initialParams? I know we have anyway from the loader but did this cause cache issues with the key?
Vitalsine85
changed the title
Vitalsine85
force-pushed
the
vital/eng-5698-update-relic-points-on-portal
branch
from
0eb46ac to
de82b4f
Compare
11 tasks
jonathanprozzi
approved these changes
Feb 6, 2025
Vitalsine85
added a commit
that referenced
this pull request
Feb 6, 2025
## Affected Packages Apps - [ ] data populator - [ ] portal - [ ] template - [x] launchpad Packages - [ ] 1ui - [ ] api - [x] graphql - [ ] protocol - [ ] sdk Tools - [ ] tools ## Overview Note: This is based off of #1092 so do not merge until that is in and this branch is rebased. Scaffolds Home/Dashboard with updated design based on Koncepted handoff. Made some general changes to the overall layout based on their designs as well, namely in the sidebar. This uses real data for the aggregated points value, but we still don't have timeseries data to be able to update the values in the cards below the aggregated IQ. That will get done in another pass when that data is ready and we are polishing. ## Screen Captures  ## Declaration - [x] I hereby declare that I have abided by the rules and regulations as outlined in the [CONTRIBUTING.md](https://github.com/0xIntuition/intuition-ts/blob/main/CONTRIBUTING.md)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Affected Packages
Apps
Packages
Tools
Overview
NOTE: Do not merge this in before the temporary activity fix, this is based on that branch, sorry.
Pulls in relic points and protocol fees for protocol points on the rewards page, in addition to general point aggregates. Eventually we will use the points events table, but in the meantime we need a temporary solution, and while this is fragmented it works. We just can't get a leaderboard at the moment.
Screen Captures
If applicable, add screenshots or screen captures of your changes.
Declaration