search

haushinka2dx / atmosphere-web

1 stars 2 forks source link

Fix not related message shown #60

Closed haushinka2dx closed 10 years ago

haushinka2dx commented 10 years ago

メッセージ通知を受けて各タイムラインにメッセージを追加する際に、タイムラインに関係のないメッセージまで表示されてしまうバグを修正しました。

具体的には、

なお、以下のPRがあたっていることが前提です。

Add parameter message ids even non search api https://github.com/haushinka2dx/atmosphere/pull/54

kannokanno commented 10 years ago

55 の件かな

kannokanno commented 10 years ago

関係ない #59 の変更も含まれていますね

haushinka2dx commented 10 years ago

そうっちゃそうですがまた微妙に違う。 ただ、結果は同じかも

kannokanno commented 10 years ago

そうっちゃそうですがまた微妙に違う。

なるほど

kannokanno commented 10 years ago

良さそうに見えます

haushinka2dx commented 10 years ago

そうか…。マスターブランチにプルリクしちゃったのでこう出ちゃうんですね。

kannokanno commented 10 years ago

のどちらかだと思いますが、前者はそのプルリクを取り込む前にremovable-timeline取り込んで閉じたら無効になってしまうので後者がスマートだと思います。

haushinka2dx commented 10 years ago

なかなか難しいですね。一つ前のに入れちゃえば済むけど単位が大きくなっちゃうし。

haushinka2dx commented 10 years ago

こちらに関して言うと完全に独立して書けるものでしたね…。

haushinka2dx commented 10 years ago

61 でマージされたのでこちらはマージせずにClose