Skip to content

Release hot-fixes for Causes#2102

Merged
ae2079 merged 9 commits intomasterfrom
staging
Aug 7, 2025
Merged

Release hot-fixes for Causes#2102
ae2079 merged 9 commits intomasterfrom
staging

Conversation

@ae2079
Copy link
Collaborator

@ae2079 ae2079 commented Aug 6, 2025

Please merge it after #2101

kkatusic and others added 5 commits August 7, 2025 02:26
* Improving evaluation cron job

* Refactor project evaluation query to aggregate category data and improve balance checking logic

* improve comment :)

* Enhance project evaluation logic by adding cause address to query and updating balance check to use cause address instead of project address

* Remove project address from project evaluation query to streamline data retrieval and focus on cause address.

* Add project address creation and deletion in project evaluation tests

---------

Co-authored-by: ali <aliebrahimi2079@gmail.com>
- Updated donation resolver to include swap transaction data in the donation fetch query.
- Enhanced GraphQL query to retrieve swap transaction details such as token symbols, amounts, and transaction hashes.
- Added a test case to verify the correct retrieval of swap transaction data associated with donations.
…vers

- Updated the handling of fromTokenAmount to ensure it is only converted to a number if it is defined and not null, improving robustness in donation creation.
- Applied the same change in both donationResolver and draftDonationResolver for consistency.
…kip unevaluated causes (#2100)

- Added tests to ensure projects with a causeScore of 0 are included in eligible projects.
- Implemented logic in `runCauseDistributionJob` to skip distribution for causes that have not been evaluated (all projects have causeScore of 0).
- Enhanced logging for skipped distributions.
- Updated the test case for donations with swap information to include cleanup logic, ensuring that created entities are removed after the test execution to prevent data pollution.
Copy link
Collaborator

@kkatusic kkatusic left a comment

Choose a reason for hiding this comment

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

LGTM ;)

@ae2079 ae2079 merged commit 49de8d9 into master Aug 7, 2025
14 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