Skip to content

Conversation

@fullmetalholder
Copy link
Contributor

feat: Improve data typing and validation for Yield Analysis SDK

This PR introduces enhancements to the data typing and validation mechanisms within the Yield Analysis SDK. Key changes include:

  • Refined Type Definitions: Updated and clarified various data types in yield_analysis_sdk/type.py to ensure better consistency and accuracy across the SDK.
  • Enhanced Validation Logic: Improvements in yield_analysis_sdk/validators.py to provide more robust and precise data validation, reducing potential errors and improving data integrity.

These changes aim to improve the overall reliability, maintainability, and usability of the SDK by strengthening its foundational data structures and validation processes.

@fullmetalholder fullmetalholder marked this pull request as draft September 4, 2025 11:01
Copy link
Contributor

@web3nerdy web3nerdy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have 2 kinds of duplicated address mixin for now.
I think we have to make it one as you created - AddressValidatorMixin.

  • remove VaultAddressValidatorMixin and replace with AddressValidatorMixin
  • change the fileld name of vault_address to address accordingly
  • change the filed name of vault_name to name as well

@web3nerdy web3nerdy marked this pull request as ready for review September 4, 2025 12:26
@web3nerdy web3nerdy merged commit 248522b into Logarithm-Labs:main Sep 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants