Project

General

Profile

Actions

提案 #1299

closed

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

Added by Kazuki Yagi over 6 years ago. Updated about 6 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
display_requirements

Description

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

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


Files

Actions #1

Updated by toshi_a 初音 over 6 years ago

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

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

Actions #2

Updated by cob odo over 6 years 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しました。確認をお願いします。

Actions #3

Updated by Kazuki Yagi over 6 years ago

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

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

Actions #4

Updated by toshi_a 初音 about 6 years ago

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

merged.

Actions

Also available in: Atom PDF