search

redmica / redmica_ui_extension

This plugin adds useful UI improvements to RedMica.
GNU General Public License v2.0
41 stars 12 forks source link

Change the default enabled setting value to true #11

Closed ishikawa999 closed 3 years ago

ishikawa999 commented 3 years ago

背景:

既存の仕組みではinit.rbで settings default: {'searchable_selectbox' => {'enabled' => 1}, 'burndown_chart' => {'enabled' => 1}} と書いていてデフォルトが有効状態になっているように見えますが、
これはSettingレコードが無いときに入れる値なので、すでに利用を始めた状態で新しく機能を追加した場合はここに 'new_feature' => {'enabled' => 1}と書いていても何もデータベースに入りません。
設定がない場合は有効扱いしてほしいという要望があがったため、対応。

設定がない場合は有効扱いするために、値が存在したら有効という現在の判定を

のように変更しました。
また、現在は無効設定にしたとき値がnilになるため、hidden_tagを足すことでチェックを外したときに0が入るように変更しています。

(今後はデフォルト無効にしたい機能が出てくるかもしれませんが、現時点ではデフォルトで有効にしたくないような機能を入れる予定が無いためこのような実装にしています。必要になったら直します。)

レビュワーには @yui-har さんと @takenory さんを指定していますが、お二人のうちお一人に見ていただいたらマージしようかなと思っています。

ishikawa999 commented 3 years ago

@yui-har ご確認いただきありがとうございました。テスト名を修正したためこれでマージします