fix: resolve TotalInvested showing zero and fix 12 UI/UX issues#787
Merged
dangershony merged 1 commit intomainfrom Apr 27, 2026
Merged
fix: resolve TotalInvested showing zero and fix 12 UI/UX issues#787dangershony merged 1 commit intomainfrom
dangershony merged 1 commit intomainfrom
Conversation
- Fix TotalInvested showing 0 for both invest and fund project types by using InvestmentRecord.InvestedAmountSats as fallback when the indexer returns TotalAmount == 0, and falling through to the handshake path to compute amount from transaction hex when needed - Add TotalInvestors to InvestedProjectDto, populated from indexer stats - Add null guard in DocumentProjectService.LatestFromNostrAsync - Standardize date format to 'dd MMM yyyy' across all user-facing dates - Add spinner/disable feedback to Settings refresh buttons - Add debug mode toggle spacing fix - Wire Edit Project button in ManageProjectContentView - Wire Deploy success 'Complete Profile' to open edit profile - Add ManageProject initial load spinner (IsRefreshing) - Add persistent refresh button in InvestmentDetailView nav bar - Skip redundant password modal for View Private Keys - Add TotalInvested > 0 assertions in both InvestAndRecoverTest and FundAndRecoverTest integration tests
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
TotalAmountcan be 0 due to lag; now falls back toInvestmentRecord.InvestedAmountSats(stored locally at publish time) and the handshake transaction hex path.InvestedProjectDto, populated from indexer stats (InvestorCount), mapped through to the portfolio detail view.dd MMM yyyy(e.g. "15 Jan 2026") across all user-facing dates (Shell, Funders, Portfolio).InitialLoadAsync()withIsRefreshingspinner.LatestFromNostrAsyncwhen project data is null.TotalInvested > 0assertions in bothInvestAndRecoverTest(invest-type) andFundAndRecoverTest(fund-type) to validate the fix.Test Results
InvestAndRecoverTest: passed (TotalInvested=0.01980000)FundAndRecoverTest: passed (TotalInvested=0.01980000)