Conversation
New budget now creates a budget for the currently selected month and auto copies the most recent budget. The dialog is no longer used.
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f5c4e5c222
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| mostRecentDate = query.Where(p => p.BudgetDate.Month < SelectedBudgetMonth.Month && p.BudgetDate.Year <= SelectedBudgetMonth.Year) | ||
| .OrderByDescending(p => p.BudgetDate) |
There was a problem hiding this comment.
The predicate here decomposes dates into Month and Year (Month < selectedMonth && Year <= selectedYear), which breaks across year boundaries. For example, when creating January 2026 with an existing December 2025 budget, December is incorrectly excluded (12 < 1 is false), so the new budget is created without copying the latest prior budget data. This also affects the “copy from” hint path that uses the same predicate, so users can be told there is nothing to copy when there actually is.
Useful? React with 👍 / 👎.
1 participant
The budget page layout has been redesigned. The list of budgets on the left side has been removed. Now budget switching is done by clicking the budget month in the header, which opens a popup where a month can be selected. Now only the budget items scroll, instead of the whole page. The header is sticky, so that the budget month and the amount left to budget are always shown, and only the budget items are scrolled.