fix: track transaction entry order and month activity#77
Conversation
![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: 364f7d0e02
ℹ️ 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".
| const summary: MonthActivitySummary = { | ||
| month: selectedMonth, | ||
| occurredAt: new Date().toISOString(), |
There was a problem hiding this comment.
recordMonthActivity() always writes month: selectedMonth, but the expense form still allows saving any payload.date. If a user is viewing March and adds or edits an expense dated April, getExpensesByMonth() will move the record to April while the new activity summary is persisted under March, leaving March showing an April transaction and April with no matching activity. This summary key needs to come from the transaction date (and edits that move months may need to update both months), not the currently viewed month.
Useful? React with 👍 / 👎.
1 participant
No description provided.