Merge branch 'fix/remote-follow-errormessage' into 'develop'

Remote follow: don't show confusing error if already following

See merge request pleroma/pleroma!267
This commit is contained in:
lambda 2018-08-12 10:55:53 +00:00
commit 9de6145737

View file

@ -99,6 +99,10 @@ def do_remote_follow(conn, %{
conn conn
|> render("followed.html", %{error: false}) |> render("followed.html", %{error: false})
else else
# Was already following user
{:error, "Could not follow user:" <> _rest} ->
render(conn, "followed.html", %{error: false})
_e -> _e ->
conn conn
|> render("follow_login.html", %{ |> render("follow_login.html", %{
@ -117,6 +121,11 @@ def do_remote_follow(%{assigns: %{user: user}} = conn, %{"user" => %{"id" => id}
conn conn
|> render("followed.html", %{error: false}) |> render("followed.html", %{error: false})
else else
# Was already following user
{:error, "Could not follow user:" <> _rest} ->
conn
|> render("followed.html", %{error: false})
e -> e ->
Logger.debug("Remote follow failed with error #{inspect(e)}") Logger.debug("Remote follow failed with error #{inspect(e)}")