Actions
提案 #1262
closedphoto_supportの改善
プラグイン名:
photo_support
Description
- Fotologの画像が展開できなくなっていたので正規表現を修正
- OpenGraphProtocol(OGP)の
og:image
で取得するコードを共通化- ついでにOGP取得コードとTwitter Card取得コードを共通化
- OGP対応サイトとしてマシュマロ https://marshmallow-qa.com/ のdefimageopenerを追加
というようなパッチを書いたので添付します。
追加したメソッド名には改善の余地があります。
Files
Updated by cob odo about 6 years ago
- File photo_support_ogp.patch photo_support_ogp.patch added
追加でpeingの対応をしようとして気づきましたが、現在のTwitter Cardでは twitter:image:src
という属性は存在せず、 twitter:image
で取得すべきかもしれません。
互換性を考え、まず twitter:image
で取得し、見つからなければ twitter:image:src
でフォールバックするように d250g2
メソッドを改修したパッチを添付します。(peing対応も含まれています)
複数の目的が混ざったパッチになってしまっているので、分離したほうがよければ分離します。
Updated by toshi_a 初音 about 6 years ago
- Status changed from 新規 to 実装待ち
- Assignee set to cob odo
ブランチ作って、commitしてもらっていいですよ。
複数の目的が混ざったパッチになってしまっているので、分離したほうがよければ分離します。
確かに分離している方が丁寧ですが、いいかなという気もします。マシュマロについては知りませんでしたが、俺はTwitterやってないのでなんとも言えませんね……。
追加したメソッド名には改善の余地があります。
めっちゃいいと思いますよ!
Updated by cob odo about 6 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee changed from cob odo to toshi_a 初音
topic/1262-extend-photo-supportブランチにpushしました。
担当はとりあえずtoshi_aさんにしておきます。よろしくお願いします。
Updated by cob odo about 6 years ago
- Tracker changed from 機能 to 提案
- Start date deleted (
2018-06-11) - ブランチ set to topic/1262-extend-photo-support
Updated by cob odo about 6 years ago
- Status changed from レビュー待ち to マージ待ち
一旦はレビューしてもらっているということでマージ待ちへ
Actions