Skip to content

Commit

Permalink
Correctly check for the current page in request breadcrumbs
Browse files Browse the repository at this point in the history
Previously, it would only match on URLs like `/request/show/14`, but not
on `/request/show/14/request_action/26`. The second URL example is what
we get whenever selecting an action in the `Select Action` dropdown of
the request#show page with the `request_show_redesign` feature toggle.

Now, breadcrumbs are displayed for both URLs.
  • Loading branch information
Dany Marcoux committed Jan 19, 2023
1 parent 12fdb5d commit 64d8b69
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
- if current_page?(request_show_path)
- if current_page?(controller: 'webui/request', action: 'show', request_action_id: params[:request_action_id])
%li.breadcrumb-item.active{ 'aria-current' => 'page' }
Request #{@bs_request.number}

0 comments on commit 64d8b69

Please sign in to comment.