Project

General

Profile

提案 #1299

display_requirementsプラグインをtwitter依存にする

Added by Kazuki Yagi 12 months ago. Updated 11 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
display_requirements
ブランチ:
topic/1299-who-requires-display-requirements

Description

display_requirementsプラグインに関してですが、twitterの規約に準拠しなければならないのはあくまでもtwitterアカウントが動くのが前提、と考えるとそれ以外の環境であればロードする必要はないのではないか?と思いましたのでパッチを作成しました。

本来であればtwitterプラグイン内にまとめるべきなのかも…とは思いましたが、例のアレのことも考えるとこれがひとまずベターな選択なのかと思われます。


Files

Associated revisions

Revision fcf10d18 (diff)
Added by cob odo 11 months ago

display_requirementsプラグインをtwitter依存にする refs #1299

History

#1

Updated by toshi_a 初音 11 months ago

  • Status changed from 分類待ち to パッチ適用待ち

Twitterを利用しない環境ではDisplay Requirementsに対応する必要がないので妥当な提案だと思います。

#2

Updated by cob odo 11 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Kazuki Yagi
  • ブランチ set to topic/1299-who-requires-display-requirements

topic/1299-who-requires-display-requirements ブランチにpushしました。確認をお願いします。

#3

Updated by Kazuki Yagi 11 months ago

  • Status changed from レビュー待ち to マージ待ち

パッチが正常に適用されている(twitterプラグインがない状態ではdisplay_requirementsがロードされない)ことを確認しました。

#4

Updated by toshi_a 初音 11 months ago

  • Status changed from マージ待ち to 終了

merged.

Also available in: Atom PDF