プロジェクト

全般

プロフィール

操作
リンクをコピー

提案 #1299

完了

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

Kazuki Yagi さんが6年以上前に追加. 6年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
Kazuki Yagi
対象バージョン:
3.8
プラグイン名:
display_requirements
ブランチ:
topic/1299-who-requires-display-requirements

説明

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

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

ファイル

display_requirements-with-twitter.patch (383 Bytes) display_requirements-with-twitter.patch Kazuki Yagi, 2018-09-29 21:19
操作
リンクをコピー
 #1

toshi_a 初音 さんが6年以上前に更新

  • ステータス を 分類待ち から パッチ適用待ち に変更

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

操作
リンクをコピー
 #2

cob odo さんが6年以上前に更新

  • ステータス を パッチ適用待ち から レビュー待ち に変更
  • 担当者 を Kazuki Yagi にセット
  • ブランチ を topic/1299-who-requires-display-requirements にセット

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

操作
リンクをコピー
 #3

Kazuki Yagi さんが6年以上前に更新

  • ステータス を レビュー待ち から マージ待ち に変更

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

操作
リンクをコピー
 #4

toshi_a 初音 さんが6年以上前に更新

  • ステータス を マージ待ち から 終了 に変更

merged.

操作
リンクをコピー

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