feat: Display SEND amounts for tasks and multipliers in rewards screen #2417
Conversation
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
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Why: Enable developers to override Supabase service ports via environment variables while maintaining sensible defaults (54321-54326). This allows running multiple Supabase instances or avoiding port conflicts without modifying Tiltfile directly. Changes: - Tiltfile: Pass SUPABASE_*_PORT env vars to supabase service - package.json: Add port defaults to test script invocation Test plan: - Command 1: tilt up Expected: Supabase starts with default ports 54321-54326 - Command 2: SUPABASE_API_PORT=55321 tilt up Expected: Supabase API runs on port 55321 instead of 54321 - Command 3: yarn workspace @my/supabase test Expected: Tests run against default ports Co-Authored-By: Warp <agent@warp.dev>
Display calculated SEND amounts earned for each task and multiplier impact in the rewards activity screen cards. Provides transparency for users on how rewards are calculated. Changes: - Add calculateTaskSendAmount() and calculateMultiplierImpact() functions - Extract buildVerificationValueMap() helper to eliminate duplication - Fix multiplier impact to calculate difference (WITH - WITHOUT) - Add comprehensive unit tests (54 tests) for calculation helpers - Refactor distributorv2.ts to use calculateSlashPercentage helper - Extract constants and hooks for better code organization - Restore database-generated.types.ts (was accidentally cleared) Tests: 54/54 passing in fixed-pool-calculation.test.ts Lint: 0 errors (biome) Co-Authored-By: Warp <agent@warp.dev>
youngkidwarrior
force-pushed
the
display-send-amounts-in-rewards
branch
from
b2c3264 to
232517b
Compare
youngkidwarrior
changed the base branch from
graphite-base/2417
to
add-percentile-calculation-helper
This was referenced Dec 30, 2025
youngkidwarrior
changed the title
|
Vercel Unique URL: https://sendapp-6y6fmwr8x-0xsend.vercel.app |
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.
feat: Display SEND amounts for tasks and multipliers in rewards screen
Display calculated SEND amounts earned for each task and multiplier impact
in the rewards activity screen cards. Provides transparency for users on
how rewards are calculated.
Changes:
Tests: 54/54 passing in fixed-pool-calculation.test.ts
Lint: 0 errors (biome)
Co-Authored-By: Warp agent@warp.dev