Project

General

Profile

Actions
Copy link

提案 #1299

closed

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

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

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

Description

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

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

Files

display_requirements-with-twitter.patch (383 Bytes) display_requirements-with-twitter.patch Kazuki Yagi, 2018-09-29 21:19
Actions
Copy link
 #1

Updated by toshi_a 初音 over 6 years ago

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

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

Actions
Copy link
 #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
Copy link
 #3

Updated by Kazuki Yagi over 6 years ago

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

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

Actions
Copy link
 #4

Updated by toshi_a 初音 about 6 years ago

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

merged.

Actions
Copy link

Also available in: Atom PDF