-
Notifications
You must be signed in to change notification settings - Fork 170
Fix issue with 30d volume not being removed, and commission not being capped #3250
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.
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
Merged
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
9a95519
Fix issue with 30d volume not being removed, and commission not being…
jusbar23 609303e
Update sliding affiliates attribution and move to process_single_match
jusbar23 6f51983
Codegen
jusbar23 ead7040
Fix proto
jusbar23 fbc9263
Lint
jusbar23 5511984
Fix lint
jusbar23 94192b4
Remove unused code
jusbar23 1cd462a
Add affiliate stuff
jusbar23 87af44e
Rewire some functions
jusbar23 16154c5
Add removal for attributedVolume
jusbar23 e03b89f
Add test case for process/expire stats
jusbar23 1fd6d5c
indexer build
jusbar23 3a66eb7
Lint
jusbar23 cf2de19
Build ecr
jusbar23 1d8ba95
Add ecr
jusbar23 ffe7545
Fix test
jusbar23 276624e
Update
jusbar23 9278e3d
Add grpc observability
jusbar23 b912cf9
Add grpc test for attributed volume
jusbar23 457a9c5
Refactor with enum
jusbar23 3ec43ac
Add referred volume
jusbar23 435b195
Update indexer protos
jusbar23 5d14a11
Fix tests
jusbar23 89217cb
Force new build
jusbar23 00cda94
Deprecate whole staked coin
jusbar23 b397dca
Format and lint
jusbar23 d8c14f2
Remove broken test cases
jusbar23 d517acd
Add error for invalid enum
jusbar23 bb75b96
Remove git webhooks
jusbar23 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical: Proto verification failures indicate API mismatch.
All three pipeline failures are from
build:verify-protodetecting that the generated TypeScript doesn't match the protobuf definition. This indicates the TypeScript code is missing fields that exist in the proto file.🧰 Tools
🪛 GitHub Actions: Indexer API Documentation Check
[error] 1-1: Build failed: 'pnpm run build:verify-proto' failed in the v4-protos package during recursive build. Exit code 1. Command: 'pnpm run build:verify-proto && rm -rf build/ && tsc'.
🪛 GitHub Actions: Indexer Build, Test & Coverage
[error] 1-1: Build failed in package 'v4-protos'. Command 'pnpm run build:verify-proto' exited with code 1. ELIFECYCLE: Command failed with exit code 1 during recursive build.
🪛 GitHub Actions: Protobuf
[error] 1-1: build:verify-proto failed. Command 'pnpm run build:proto' exited with code 1 in workspace '@dydxprotocol-indexer/v4-protos'. Proto transpilation produced diffs indicating API changes that require validation (see logs for details).
🤖 Prompt for AI Agents