Remove "most recent notification" endpoint.
This commit is contained in:
parent
6ac5d48575
commit
a6812bc9df
|
@ -265,12 +265,6 @@ defmodule Pleroma.Web.Router do
|
||||||
post("/account/update_profile_banner", TwitterAPI.Controller, :update_banner)
|
post("/account/update_profile_banner", TwitterAPI.Controller, :update_banner)
|
||||||
post("/qvitter/update_background_image", TwitterAPI.Controller, :update_background)
|
post("/qvitter/update_background_image", TwitterAPI.Controller, :update_background)
|
||||||
|
|
||||||
post(
|
|
||||||
"/account/most_recent_notification",
|
|
||||||
TwitterAPI.Controller,
|
|
||||||
:update_most_recent_notification
|
|
||||||
)
|
|
||||||
|
|
||||||
get("/statuses/home_timeline", TwitterAPI.Controller, :friends_timeline)
|
get("/statuses/home_timeline", TwitterAPI.Controller, :friends_timeline)
|
||||||
get("/statuses/friends_timeline", TwitterAPI.Controller, :friends_timeline)
|
get("/statuses/friends_timeline", TwitterAPI.Controller, :friends_timeline)
|
||||||
get("/statuses/mentions", TwitterAPI.Controller, :mentions_timeline)
|
get("/statuses/mentions", TwitterAPI.Controller, :mentions_timeline)
|
||||||
|
|
|
@ -350,20 +350,6 @@ def external_profile(%{assigns: %{user: current_user}} = conn, %{"profileurl" =>
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def update_most_recent_notification(%{assigns: %{user: user}} = conn, %{"id" => id}) do
|
|
||||||
with id when is_number(id) <- String.to_integer(id),
|
|
||||||
info <- user.info,
|
|
||||||
mrn <- max(id, user.info["most_recent_notification"] || 0),
|
|
||||||
updated_info <- Map.put(info, "most_recent_notification", mrn),
|
|
||||||
changeset <- User.info_changeset(user, %{info: updated_info}),
|
|
||||||
{:ok, _user} <- User.update_and_set_cache(changeset) do
|
|
||||||
conn
|
|
||||||
|> json_reply(200, Jason.encode!(mrn))
|
|
||||||
else
|
|
||||||
_e -> bad_request_reply(conn, "Can't update.")
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def followers(conn, params) do
|
def followers(conn, params) do
|
||||||
with {:ok, user} <- TwitterAPI.get_user(conn.assigns[:user], params),
|
with {:ok, user} <- TwitterAPI.get_user(conn.assigns[:user], params),
|
||||||
{:ok, followers} <- User.get_followers(user) do
|
{:ok, followers} <- User.get_followers(user) do
|
||||||
|
|
|
@ -31,26 +31,6 @@ test "with credentials", %{conn: conn, user: user} do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe "POST /api/account/most_recent_notification" do
|
|
||||||
setup [:valid_user]
|
|
||||||
|
|
||||||
test "without valid credentials", %{conn: conn} do
|
|
||||||
conn = post(conn, "/api/account/most_recent_notification.json")
|
|
||||||
assert json_response(conn, 403) == %{"error" => "Invalid credentials."}
|
|
||||||
end
|
|
||||||
|
|
||||||
test "with credentials", %{conn: conn, user: user} do
|
|
||||||
conn =
|
|
||||||
conn
|
|
||||||
|> with_credentials(user.nickname, "test")
|
|
||||||
|> post("/api/account/most_recent_notification.json", %{id: "200"})
|
|
||||||
|
|
||||||
assert json_response(conn, 200)
|
|
||||||
user = User.get_by_nickname(user.nickname)
|
|
||||||
assert user.info["most_recent_notification"] == 200
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe "POST /statuses/update.json" do
|
describe "POST /statuses/update.json" do
|
||||||
setup [:valid_user]
|
setup [:valid_user]
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue