Implement fetch-messages - Githubissues

vim-jp / slacklog-generator

Archive generator for Slack

https://vim-jp.org/slacklog/

Creative Commons Attribution 4.0 International

26 stars 9 forks source link

Implement fetch-messages #138

Closed kyoh86 closed 4 years ago

kyoh86 commented 4 years ago

概要

KoRoNさんが途中まで書き上げていたfetch-messagesのPR

https://github.com/vim-jp/slacklog-generator/pull/116

を引き継ぐ形で実装してみました。

私の理解 = やや不安なところ

internal/slackadapterの実装方針の理解があっているかどうかに少し不安が。（動くのでリファクタリングレベルでの話である）

責務は「使いやすい param / response で公式ライブラリをラップ」
- 使いにくい定義のWrap
- timestamp string <=> *time.Time
- Cursor 処理の隠蔽
- やや冗長な書き味の箇所が含まれるので、もう少しSmartな定義にしたかったかも
  - internal/slackadapter/conversations_history.go#L33-38とか
- クライアントの生成も各関数の中で実施する
- slack.Newを裸でそのまま各関数に書かせるのは少しリスクかも
- NewしたClientを持ち回すのとどっちが良いかは見えていない

試し方

$ go run . fetch-messages --data-dir ./_logdata/slacklog_data --date 2020-07-05 --verbose