Actions
バグ #1343
closedMastodonプラグインが使っているactivityタイプを、専用のものにするべき
プラグイン名:
クラッシュする:
No
Description
Mastodonプラグインが発生させるアクティビティメッセージのうち、systemタイプになっているものを別のタイプに修正する。
現在は全てがsystemになっており、Mastodonに関する通知をユーザが制御できない状態になっている
Files
Updated by Shibafu Midorino over 5 years ago
- File 0001-mastodon-system.patch 0001-mastodon-system.patch added
- Status changed from 実装待ち to パッチ適用待ち
現在 system activity が使用されているのは3箇所でしょうか。
[shibafu@lulua ~/git/mikutter]$ rg ":system" core/plugin/mastodon core/plugin/mastodon/api.rb 124: Plugin.activity(:system, "APIアクセス失敗", description: "URI: #{uri}\nparameters: #{params}\nHTTP status: #{resp.status}\nresponse:\n#{resp.body}") rescue nil core/plugin/mastodon/mastodon.rb 49: activity(:system, "自分のプロフィールやフォロー関係を取得しています...") 56: activity(:system, "自分のプロフィールやフォロー関係の取得が完了しました(#{world.account.acct})")
そのうち後者については、worldon時代に個人的に使用していたパッチがありますので提供します。
前者は、このようなケースの場合ってactivityで出すべきなんでしょうかね…?
Updated by toshi_a 初音 over 5 years ago
- Assignee set to toshi_a 初音
- ブランチ set to topic/1343-mastodon-activity-kind
ブランチ作成しましたが、俺も変更入れるのでもう少し待ってください(後述)
前者は、このようなケースの場合ってactivityで出すべきなんでしょうかね…?
いえ、ないです。これを消すのは俺の方でやります。
初期のmikutterはTwitterがレスポンスを返してくることは稀であるという考え方で一切が設計されているからです。
例外として、ユーザの操作によって能動的に取得される場合は失敗時のフィードバックがあったほうが良いですが、APIリクエストを行うレイヤーではこの判断は出来ません。
Updated by toshi_a 初音 over 5 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee changed from toshi_a 初音 to Shibafu Midorino
対応しました。確認してもらって良いですか。
Updated by Shibafu Midorino over 5 years ago
- Status changed from レビュー待ち to マージ待ち
確認しました。問題なく動作しています。
Actions