Skip to content

Commit

Permalink
Rename deprecated push_redirect to push_navigate (#444)
Browse files Browse the repository at this point in the history
Removes compile time deprecation warnings
  • Loading branch information
joshprice authored Jun 21, 2024
1 parent f87a1bd commit 5fe566b
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions lib/phoenix/live_dashboard/page_live.ex
Original file line number Diff line number Diff line change
Expand Up @@ -288,7 +288,7 @@ defmodule Phoenix.LiveDashboard.PageLive do

if node && node != page.node do
to = PageBuilder.live_dashboard_path(socket, page.route, node, page.params, page.params)
{:noreply, push_redirect(socket, to: to)}
{:noreply, push_navigate(socket, to: to)}
else
{:noreply, redirect_to_current_node(socket)}
end
Expand Down Expand Up @@ -366,7 +366,7 @@ defmodule Phoenix.LiveDashboard.PageLive do
end

defp redirect_to_current_node(socket) do
push_redirect(socket, to: PageBuilder.live_dashboard_path(socket, :home, node(), %{}, %{}))
push_navigate(socket, to: PageBuilder.live_dashboard_path(socket, :home, node(), %{}, %{}))
end

defp update_page(socket, assigns) do
Expand Down
4 changes: 2 additions & 2 deletions lib/phoenix/live_dashboard/pages/metrics_page.ex
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ defmodule Phoenix.LiveDashboard.MetricsPage do
cond do
nav && is_nil(metrics) ->
to = live_dashboard_path(socket, socket.assigns.page, nav: first_nav)
{:ok, push_redirect(socket, to: to)}
{:ok, push_navigate(socket, to: to)}

metrics && connected?(socket) ->
Phoenix.LiveDashboard.TelemetryListener.listen(socket.assigns.page.node, metrics)
Expand All @@ -27,7 +27,7 @@ defmodule Phoenix.LiveDashboard.MetricsPage do

first_nav && is_nil(nav) ->
to = live_dashboard_path(socket, socket.assigns.page, nav: first_nav)
{:ok, push_redirect(socket, to: to)}
{:ok, push_navigate(socket, to: to)}

true ->
{:ok, assign(socket, metrics: nil, nav: nav)}
Expand Down
2 changes: 1 addition & 1 deletion lib/phoenix/live_dashboard/pages/request_logger_page.ex
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ defmodule Phoenix.LiveDashboard.RequestLoggerPage do
def mount(_, %{request_logger: _}, socket) do
stream = :crypto.strong_rand_bytes(3) |> Base.url_encode64()
to = live_dashboard_path(socket, socket.assigns.page, stream: stream)
{:ok, push_redirect(socket, to: to)}
{:ok, push_navigate(socket, to: to)}
end

@impl true
Expand Down

0 comments on commit 5fe566b

Please sign in to comment.