Skip to content
This repository was archived by the owner on May 31, 2023. It is now read-only.

自分の投稿したレビュー一覧をチャットに表示する #352

Merged
merged 5 commits into from
Jun 15, 2022

Conversation

takuya-okamoto-esm
Copy link
Contributor

@takuya-okamoto-esm takuya-okamoto-esm commented Jun 14, 2022

#162

Slack Appを設定しているチャンネルで「myreview」と発言すると自分の投稿したレビュー一覧を表示してみました。
本番環境だと #bee で発言すると動くはずです。

image

@takuya-okamoto-esm takuya-okamoto-esm added the カイゼン ユーザーに直接価値は与えないもの。リファクタやシステム構築。 label Jun 14, 2022
@takuya-okamoto-esm takuya-okamoto-esm self-assigned this Jun 14, 2022
@takuya-okamoto-esm
Copy link
Contributor Author

Bee Demo Appの環境で動作確認できました。

image

@takuya-okamoto-esm
Copy link
Contributor Author

見栄えをちょっとだけ調整してみました。

image

@takuya-okamoto-esm
Copy link
Contributor Author

まだ投稿していない場合のメッセージを追加

image

@takmita
Copy link
Contributor

takmita commented Jun 15, 2022

修正不要です。
タイトルの頭に、「・」があると見やすそうだと思いました。

client.chat_postEphemeral(
channel=os.environ["NOTIFY_POST_REVIEW_CHANNEL"],
user=user_id,
text=f"<@{user_id}>さんが投稿したレビューはこれです({num_reviews}冊)\n\n {myreviews}",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感想です。
個人の捉え方次第なんですが、表示されているのがそのユーザが投稿した本の一覧なので、
〇〇さんがレビュー投稿した本の一覧ですみたいな方が、いいのかなと思いました。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確かに。
ちょっと変えてみました。
image

Copy link
Contributor

@hirokisakabe hirokisakabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コマンドは可能ならば、bee <サブコマンド>が良いかなと思いました。bee myreviewとか
他のアプリとコンフリクトしないように、メインのコマンドはbee一つが良いと思います。

ただ、すぐ対応できなければ、このままでも良いです 👍

@takuya-okamoto-esm
Copy link
Contributor Author

コマンドは可能ならば、bee <サブコマンド>が良いかなと思いました。

ちょっと考える時間が欲しいので、やるときは別Issudeでやりたいと思います。

今回コマンドを作るときにマニュアルとかヘルプコマンドもあった方が良いかな?と少し思いましたが
逆にコマンド一覧とか、サブコマンドによる整理が必要になるほど複雑なコマンドを作っちゃうと
ユーザとしては使わなくなるよなー、とも思いました。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
カイゼン ユーザーに直接価値は与えないもの。リファクタやシステム構築。
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants