第4回/Allow configuration of API limit instead of hardcoding at 100 #19
第4回
参加者: yebisさん、ながせさん、Juno NISHIZAKI
RestAPI の limit の上限が 100 で固定されているので拡張できるようにしたい
https://github.com/redmine-patch-meetup/redmine-dev-mirror/issues/19
事前調査／検討
https://github.com/redmine/redmine/blob/52cc3fdc24dbb3814b6924323c032051ddac941e/app/controllers/application_controller.rb#L660-L661
ここで上限を100件に制限してそう。
制限はIssueで提案されているとおりAdministrationの設定で変えるようにするのが良さそう
対応内容
基本的には https://www.redmine.org/issues/16069 の description の案を踏襲するが、以下の箇所を変更した
設定値のフォームは「全般」タブから「API」タブに移動した
limit が未設定や 0 未満の値が指定されたときに 25 件に強制的に置き換える処理があったので、設定値が 25 未満だったときにさらに置き換えられるように分岐処理を調整した
ラベル名が長かったので短くした
対応完了。下記プルリクで CI 実行中
https://github.com/redmine-patch-meetup/redmine-dev-mirror/pull/21
以下でパッチを投稿。
https://www.redmine.org/issues/16069#note-34