Docker/Maybe
Docker/Maybe
1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-08-05 05:25:24 +02:00
Code Issues Releases Activity Actions

Enhance account links in transfers view to handle missing accounts gracefully. Added conditional checks to display a warning message when account data is unavailable.

Browse source
This commit is contained in:
Josh Pigford 2025-04-30 14:29:33 -05:00
parent ae41b3de46
commit 47017a6432
1 changed files with 19 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

17
app/views/transfers/_account_links.html.erb
View file

@ -1,7 +1,24 @@
<%# locals: (transfer:, is_inflow: false) %>
<div class="flex items-center gap-1">
<% first_account, second_account = is_inflow ? [transfer.to_account, transfer.from_account] : [transfer.from_account, transfer.to_account] %>
<%# Check if first_account exists before creating link %>
<% if first_account %>
<%= link_to first_account.name, account_path(first_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
<% else %>
<span class="text-warning text-xs italic" title="Transfer ID: <%= transfer.id %>">
Data Error: Missing account
</span>
<% end %>
<%= lucide_icon is_inflow ? "arrow-left" : "arrow-right", class: "w-4 h-4 shrink-0" %>
<%# Check if second_account exists before creating link %>
<% if second_account %>
<%= link_to second_account.name, account_path(second_account, tab: "activity"), class: "hover:underline", data: { turbo_frame: "_top" } %>
<% else %>
<span class="text-warning text-xs italic" title="Transfer ID: <%= transfer.id %>">
Data Error: Missing account
</span>
<% end %>
</div>