プロジェクト

全般

プロフィール

機能 #1140

プロフィール更新Spellが欲しい

Shibafu Midorino10ヶ月前に追加. 9ヶ月前に更新.

ステータス:
終了
優先度:
通常
対象バージョン:
開始日:
2017-12-26
期日:
進捗率:

0%

プラグイン名:
Twitter

説明

表題の通りですが、アカウントのプロフィールを更新するためのSpell搭載の要望および提案です。

mikutterを使ってメンションに反応してプロフィールを更新するbotを運用しているのですが、
3.5.xまでは、独自で MikuTwitter::APIShortcuts にモンキーパッチしてAPIを呼び出していました。

3.6からはSpellによって正攻法でこうした拡張ができますが、Worldごとの仕様を考慮して利用側でいくつも実装するのは辛いです。

「プロフィールの更新」に絞れば、複数のサービスで対応するAPIが提供されていますので
リファレンス実装の意味も込めてTwitter Worldに搭載していただきたいと考えています。

自作プラグインの実装を土台とした、3.6.0のTwitter World向けパッチを添付しました。
Spellの分割単位は「更新可能な項目ごと」としています。
これは、サービスによって一度に変更可能な項目に差があるからです。
(例: Mastodonは1つのAPIで全ての項目を受け付けるが、Twitterは画像系が別エンドポイントになっている)

ご検討の程、よろしくお願いします。

0001-update_profile_spell.patch (1.09 KB) 0001-update_profile_spell.patch Shibafu Midorino, 2017-12-26 18:23
0002-update_profile_spell.patch (1.17 KB) 0002-update_profile_spell.patch Shibafu Midorino, 2017-12-27 23:45

関係しているリビジョン

リビジョン 75c4a9dc (差分)
Shibafu Midorino9ヶ月前に追加

プロフィール更新Spell refs #1140

履歴

#1 toshi_a 初音10ヶ月前に更新

  • 担当者Shibafu Midorino にセット

概ね賛成ですが、mergeするにあたって以下の二点は修正しておきたいです。

Spellの戻り値を twitter_user Model にする

これは単に、jsonメソッドを呼んでいるところをuserにすれば良いだけだと思います。Deferredをチェインした時に、変更後の twitter_user Modelを得られるようにしておかないと、Worldが別のものだった時に、値の構造が違いすぎて利用できないからです。

IconのアップデートにはPhoto Modelを利用する

現在、mikutter上では、画像を扱うポータブルな方法として Photo Model を使うようにしているので、アイコンの指定にもPhoto Modelを使えるようにしたいです

#2 Shibafu Midorino10ヶ月前に更新

指摘のあった2点について対応を行いました。確認をお願いします。

IconのアップデートにはPhoto Modelを利用する

こちらについては、Spell制約[twitter]であったところを[twitter, photo]と変更しています。
Modelなのでこのようにしましたが、よろしかったでしょうか?

P.S. #1143 のチケットの動向が気になります...

#3 toshi_a 初音9ヶ月前に更新

  • ステータス新規 から レビュー待ち に変更

masterブランチにmergeしました。一応確認してもらえますか

#4 Shibafu Midorino9ヶ月前に更新

  • ステータスレビュー待ち から 終了 に変更

確認できました。対応ありがとうございました。

他の形式にエクスポート: Atom PDF