You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is a missing closing brace for the conditional rendering of userStats:
{userStats&&(
...
)}
There are several instances where the Text component is missing its closing tag. Make sure to close the Text component properly.
In the Text component where you mention 'Total Users with Email', the text should be wrapped within curly braces and not angle brackets. Also, ensure the .toLocaleString() is correctly used after accessing the respective value from userStats for all instances.
In the Link component to Twitter, the text '@PushProtocol' should be wrapped in curly braces or quotation marks to make it a string.
The img tag for PushIcon should have an alt attribute for accessibility.
In the rewards.ts file:
In the UserRewardsStatsResponse type, the allSeasons object is missing its property definitions. Add the missing properties for allSeasons.
In the UserRewardsDetailParams type, there is a missing closing brace after the caip10WalletAddress property.
In the createUserRewardsDetailResponse type, it currently has an empty object as a value. Ensure this structure fits the expected response for user rewards details.
Overall, please fix the mentioned issues in the code. After the necessary corrections, please perform testing to ensure everything works as expected.
This file contains hidden or 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
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.
Pull Request Template
Ticket Number
Description
Update Stats to Vault
Type of Change
Checklist
Frontend Guidelines
Build & Testing
Screenshots/Video with Explanation
Before: Explain the previous behavior
After: What's changed now
Additional Context
Review & Approvals
Notes