Enhance investment status display #414
Conversation
… days left based on project end date
There was a problem hiding this comment.
Pull Request Overview
Enhance the investment status display by introducing distinct logic for projects that haven't started yet, handling missing end dates via stage fallbacks, and updating the days-left calculation.
- Introduce
currentDate,startDate, andinvestmentEndDatevariables - Add fallback to stage release date when
EndDateis undefined - Split status display into “days until start,” “days left,” and “after end” branches
Comments suppressed due to low confidence (1)
src/Angor/Client/Pages/View.razor:407
- [nitpick] This new branch for displaying days until start isn't covered by existing tests. Consider adding a unit test for a project with a future start date.
@if (currentDate < startDate)
…o end date is set
…for missing End Date
src/Angor/Client/Pages/View.razor
Outdated
| { | ||
| endDate = project.ProjectInfo.Stages.First().ReleaseDate; | ||
| } | ||
| throw new InvalidOperationException("Project must have a valid End Date. Projects without End Date are not supported."); |
There was a problem hiding this comment.
What actually happens when we throw in the html?
There was a problem hiding this comment.
I don't think we need to throw an exception, by this logic you would also throw an exception if start date is null.
We don't expect that t o happen
just keep this line
var endDate = project.ProjectInfo.EndDate;
Can you suggest how to handle it? |
No description provided.