Commit f87b3156 authored by lain's avatar lain
Browse files

TwitterAPI: Fix dm_timeline displaying only half of the conversation.

parent 2f639ea1
Pipeline #4507 passed with stages
in 6 minutes and 51 seconds
...@@ -130,7 +130,7 @@ def dm_timeline(%{assigns: %{user: user}} = conn, params) do ...@@ -130,7 +130,7 @@ def dm_timeline(%{assigns: %{user: user}} = conn, params) do
query = query =
ActivityPub.fetch_activities_query( ActivityPub.fetch_activities_query(
[user.ap_id], [user.ap_id],
Map.merge(params, %{"type" => "Create", visibility: "direct"}) Map.merge(params, %{"type" => "Create", "user" => user, visibility: "direct"})
) )
activities = Repo.all(query) activities = Repo.all(query)
......
...@@ -284,6 +284,12 @@ test "it show direct messages", %{conn: conn} do ...@@ -284,6 +284,12 @@ test "it show direct messages", %{conn: conn} do
"visibility" => "direct" "visibility" => "direct"
}) })
{:ok, direct_two} =
CommonAPI.post(user_two, %{
"status" => "Hi @#{user_one.nickname}!",
"visibility" => "direct"
})
{:ok, _follower_only} = {:ok, _follower_only} =
CommonAPI.post(user_one, %{ CommonAPI.post(user_one, %{
"status" => "Hi @#{user_two.nickname}!", "status" => "Hi @#{user_two.nickname}!",
...@@ -296,8 +302,9 @@ test "it show direct messages", %{conn: conn} do ...@@ -296,8 +302,9 @@ test "it show direct messages", %{conn: conn} do
|> assign(:user, user_two) |> assign(:user, user_two)
|> get("/api/statuses/dm_timeline.json") |> get("/api/statuses/dm_timeline.json")
[status] = json_response(res_conn, 200) [status, status_two] = json_response(res_conn, 200)
assert status["id"] == direct.id assert status["id"] == direct_two.id
assert status_two["id"] == direct.id
end end
end end
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment