-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add finished and unfinished Runs tabs in simulators#:id #733
Conversation
@yohm Finished / Unfinished Runsが表示されるようにしてみました。スクリーンショットはPR descriptionの通りです。 一点、Analysesも表示した方が良さそうな気がしたので、そちらの対応も必要か、確認お願いいたします 🙇♂️ |
render json: RunsListDatatable.new(Run.in(status: stat), view_context) | ||
conditions = {} | ||
conditions[:status] = params["run_status"] # required | ||
conditions[:simulator_id] = params["simulator_id"] if params["simulator_id"].present? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
optionalな引数として simulator_id
を渡せるように更新しています
$(function() { | ||
// finished runs | ||
const oFinishedRunsTable = datatables_for_runs_table('#finished_runs_list'); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この辺のloadingコードはshowの方にまとめても良かったのですが、参照しているElement idが同じファイル内に見えていた方が可読性としては良いかなと思い、それぞれのpartial側で呼び出すようにしています。
定例にて、Analysesは表示不要とのこと(Progressも同様なので、それに従う)を確認頂いたので、テストコードを増やした上でDraftを外す方向で対応させて頂きます。 |
28f466a
to
61d7f8a
Compare
@yohm こちら、テストコードの方追加いたしましたのでmerge readyになります。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#732 の変更への追従をお願いします 🙇
simulators#showページに
Finished Runs
及びUnfinished Runs
を追加 refs #717要確認