Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 7, 2025

Summary of the problem

The ledger Type filter was missing "Wise transfer" despite the WiseTransfer model and supporting infrastructure already existing in the codebase.

Describe your changes

Added "wise_transfer" to the transaction type filter:

  • View layer: Added to filter menu type array in app/views/events/_filter_menu.html.erb
  • Controller layer: Added filter logic to EventsController.filter_transaction_type method
    • Settled transactions: t.local_hcb_code.wise_transfer?
    • Pending transactions: t.wise_transfer_id

The implementation follows the existing pattern used by other transfer types (ACH, PayPal, Wire) and works with both the web UI and API v4.

Original prompt

This section details on the original issue you should resolve

<issue_title>[Ledger] Add wise transfer filter type</issue_title>
<issue_description>Image</issue_description>

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


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Luke-Oldenburg <87272260+Luke-Oldenburg@users.noreply.github.com>
Copilot AI changed the title [WIP] Add wise transfer filter type to ledger Add Wise transfer filter type to ledger Dec 7, 2025
Copilot AI requested a review from Luke-Oldenburg December 7, 2025 05:09
Copilot finished work on behalf of Luke-Oldenburg December 7, 2025 05:09
Copy link
Contributor

@Luke-Oldenburg Luke-Oldenburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@Luke-Oldenburg Luke-Oldenburg marked this pull request as ready for review December 7, 2025 05:15
@Luke-Oldenburg Luke-Oldenburg requested a review from a team December 7, 2025 05:15
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.

[Ledger] Add wise transfer filter type

2 participants