Skip to content

Fix locked balance allocation#3211

Merged
koutst merged 10 commits intomainfrom
fix-locked-balance-allocation
Mar 20, 2026
Merged

Fix locked balance allocation#3211
koutst merged 10 commits intomainfrom
fix-locked-balance-allocation

Conversation

@koutst
Copy link
Contributor

@koutst koutst commented Mar 18, 2026

This PR fixes how locked vs. unlocked RSC balance is tracked during fundraise contributions and refunds

  • Locked balance contributions were being refunded as unlocked.
  • Add purchase to balances to reference on refund.

@koutst koutst requested a review from a team as a code owner March 18, 2026 14:36
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@koutst koutst force-pushed the fix-locked-balance-allocation branch from 2108a10 to 8aa626f Compare March 18, 2026 18:49
@koutst koutst force-pushed the fix-locked-balance-allocation branch from 089fa6b to 85f289e Compare March 19, 2026 22:04
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

❌ Patch coverage is 96.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.63%. Comparing base (1191eb3) to head (85f289e).
⚠️ Report is 17 commits behind head on main.

Files with missing lines Patch % Lines
src/purchase/services/fundraise_service.py 94.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3211      +/-   ##
==========================================
+ Coverage   78.62%   78.63%   +0.01%     
==========================================
  Files         614      614              
  Lines       34415    34437      +22     
==========================================
+ Hits        27059    27081      +22     
  Misses       7356     7356              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@koutst koutst merged commit 050123f into main Mar 20, 2026
5 checks passed
@koutst koutst deleted the fix-locked-balance-allocation branch March 20, 2026 12:59
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.

1 participant