optimized the method of manage timeline

[haushinka2dx]

atmos.js のコードがかなりグチャグチャになってきたので整理した。 機能的な変更はありません。

• タイムラインを管理するコードが atmos.js にぐちゃぐちゃに存在していたので atmos-timeline-manager.js に該当ロジックを抽出。
• タイムラインの定義情報に以下を追加
  • api : タイムラインのメッセージを取得する際に使用するAPI
  • private : タイムラインがプライベートメッセージを扱うかどうかをセット

[kannokanno]

LGTM

[kannokanno]

この略称思い出すのに時間かかった。 動かしてないので全然自信はないですが、いいのではないかと思います。

今後の課題として、安心するためにやはり多少のテストが欲しいですね。Web側は面倒ではあるんですが

[haushinka2dx]

チェックありがとうございます。

実装していて bind と call が混乱しました。

今後の課題として、安心するためにやはり多少のテストが欲しいですね。Web側は面倒ではあるんですが

悲しいことにどうやれば出来るのかすら分かってないです :cry:

[kannokanno]

webdriverとか使えば出来るのは想像付いているんですが、それだと

• 普通にやるとテストを書くコストが高い
  • UIに依存し過ぎるとテストのメンテ性が極端に下がる
• 環境づくりとかの面でだるさがありそう
• CIをどうやって回すか

などの問題があって、メンテ性が低そうだなってところが悩みです。 そのうち追加したいとは思っています。 「継続的デリバリー」にそんな感じのトピックがあったはずなのでそのうち読み直したい(希望)

[haushinka2dx]

さすが色々知ってますね。 こういった情報ってIssueにまとめるべきですかね。 なんとなく埋もれさせたくないけどIssueが単なるメモになっちゃうのもちょっとなあと。

と徒然なるままに書きつつマージします。