Skip to content

Fix bulk edit transaction state by using correct field name#282

Merged
jwaspin merged 3 commits intomainfrom
copilot/fix-bulk-edit-transaction-state
Feb 17, 2026
Merged

Fix bulk edit transaction state by using correct field name#282
jwaspin merged 3 commits intomainfrom
copilot/fix-bulk-edit-transaction-state

Conversation

Copy link
Contributor

Copilot AI commented Feb 17, 2026

Bulk edit was failing to update transaction state while date and category changes worked correctly.

Root cause: BulkEditModal was setting updates.status instead of updates.transactionState, causing the Redux reducer to silently ignore the state update since the schema field is transactionState per TransactionFields.TRANSACTION_STATE.

Changes:

  • Updated BulkEditModal.jsx line 46 to use transactionState field name
  • Bumped version to 2.3.1
// Before
if (selectedState) {
  updates.status = selectedState;  // Wrong field name
}

// After
if (selectedState) {
  updates.transactionState = selectedState;  // Matches schema
}

This aligns with how date and category updates work in the same component.

Original prompt

This section details on the original issue you should resolve

<issue_title>Bulk Edit Not Working to Change State</issue_title>
<issue_description>When selecting multiple transactions, click 'edit' and select a new state, then click 'change all' and nothing happens. The date and category change as needed, but not the transaction state.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jwaspin <6432180+jwaspin@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 17, 2026

Deploying beta-lucaledger with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8181521
Status: ✅  Deploy successful!
Preview URL: https://f7cc5e8f.beta-lucaledger.pages.dev
Branch Preview URL: https://copilot-fix-bulk-edit-transa.beta-lucaledger.pages.dev

View logs

Copilot AI changed the title [WIP] Fix bulk edit functionality for changing transaction state Fix bulk edit transaction state by using correct field name Feb 17, 2026
Copilot AI requested a review from jwaspin February 17, 2026 02:28
@jwaspin jwaspin marked this pull request as ready for review February 17, 2026 02:39
@jwaspin jwaspin merged commit 3a8c1c9 into main Feb 17, 2026
2 checks passed
@jwaspin jwaspin deleted the copilot/fix-bulk-edit-transaction-state branch February 17, 2026 02:39
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.

Bulk Edit Not Working to Change State

2 participants