Actions
提案 #1262
closed
photo_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