masuilab / Gyazz

Node+Express+MongoDBによるGyazzの再実装

22 stars 1 forks source link

Closed nikezono closed 9 years ago

nikezono commented 9 years ago

191

Wiki/Page/rss.xmlで取得できるようにしました（metaタグでそう指定してあったので）。

ページの取得については、現在、更新頻度順で、タイトルの重複を許していますが、

どちらにすべきでしょうか？

shokai commented 9 years ago

動かないじゃん

nikezono commented 9 years ago

ローカルだと動くんですが・・・どういうエラーでしょう

shokai commented 9 years ago

coffeelintだ

nikezono commented 9 years ago

特にこのファイル触ってないので、masterが既に引っかかってるようです

shokai commented 9 years ago

master修正しました同じページは一つのアイテムとして扱うでいいと思います

nikezono commented 9 years ago

@shokai ありがとうございます。

Rebaseし、1つのPageを1つにまとめる様にしました。

その際、models/page#timesortメソッドに一部修正を加えたので、こちらご確認願います。

nikezono commented 9 years ago

あれ、先祖返りしてるっぽいな

shokai commented 9 years ago

models/page#timesort みたけど、これlistページがめっちゃ重くなるとかある・・かも？後で試します

nikezono commented 9 years ago

@shokai ありがとうございます。 mlistにinclude_text=falseみたいなオプションを付ける形で対応するのはどうでしょう。

shokai commented 9 years ago

それでいいと思いマーす

shokai commented 9 years ago

なんかchromeでrss見たらこんなの出た

shokai commented 9 years ago

このエラーはまあ・・大丈夫だろ

mergeできないのでrebaseしてpush -fで上書きしておいてください

nikezono commented 9 years ago

force push しました

nikezono commented 9 years ago

タイトル修正しましたこれでとりあえず問題ないかと思います