Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attachment での opengraph 他への対応 #96

Open
koron opened this issue May 10, 2020 · 1 comment
Open

Attachment での opengraph 他への対応 #96

koron opened this issue May 10, 2020 · 1 comment

Comments

@koron
Copy link
Member

koron commented May 10, 2020

#35 からの派生。

いま Attachement はフロント側で ServiceName をみて twitter や GitHub で振り分けて出してる。
しかし known なサービスすべてに独自コード書くのはアホらしいので統一した手段が欲しい。

特に ThumbURL が無効なのが災いしてて、それをどこかからか補えればいい感じに表示できるんではなかろうか? その候補としては OpenGraph (og:image) か。

どこかのタイミングで ThumbURL のないやつの FromURL (これがないのはまれ) を取ってきて、HTMLなら OpenGraph 解析して、ThumbURL 他の必須パラメータを埋め込む、という案。

あわせて Attachement について Go 側とフロント側で意識合わせしておきたいが…

どうやろか?



公式 https://api.slack.com/reference/messaging/attachments 見ると Attachement は古いから Block 使えって書いてるな…

なお layout Block はこっち https://api.slack.com/reference/block-kit/blocks

@heavenshell
Copy link
Member

フロントエンド側としては外部サービスごとの if 分岐はなくしたいです。
今現状辛いのが、Attachment でサービスごとに出してる項目がバラバラというのが辛いので、
できるだけ統一したいです。

  • サービス名
  • 外部サービスのアイコン(og:image?)
  • 本文
  • 添付(Twitter の埋め込んでる画像/動画)

個人的な意見では添付も出さなくていいのかなーと思ってます。
あれば便利なのは承知してますが、
特にスマホだと小さくなるし、見たければリンクタップして飛んでくれというお気持ちです。
(スマホで見ると Twitter の動画は小さいサムネイルしか出してない)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants