-
Notifications
You must be signed in to change notification settings - Fork 11
feat: Added support for fetchin apps from debank #247
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
base: master
Are you sure you want to change the base?
Conversation
5fa8c99 to
27a78ed
Compare
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.
Pull request overview
Adds first-class support for DeBank “complex_app_list” data (apps like Polymarket), including parsing deposits and prediction positions into typed models.
Changes:
- Added
predictiontoAssetType. - Implemented DeBank app models +
DebankAppParser, and added a new API endpoint key + fetch method forcomplex_app_list. - Added pytest coverage for parsing DeBank app responses.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| blockapi/v2/models.py | Adds AssetType.PREDICTION to represent prediction positions. |
| blockapi/v2/api/debank.py | Introduces DeBank app/prediction models and parser; adds get_complex_app_list endpoint and fetch_debank_apps. |
| blockapi/v2/api/init.py | Re-exports new DeBank app-related classes from the API package. |
| blockapi/test/v2/api/debank/test_debank_app_parser.py | Adds tests covering parsing deposits and prediction positions from complex_app_list. |
| blockapi/test/v2/api/debank/conftest.py | Adds an app_parser fixture for the new parser. |
| .vscode/settings.json | Enables pytest integration in VS Code workspace settings. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
69f90d7 to
411ad1c
Compare
|
|
||
|
|
||
| @pytest.fixture | ||
| def app_parser(): |
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.
nitpick: i prefer debank_app_parser here
acdf88c to
70b9e9b
Compare
No description provided.