Skip to content

Commit

Permalink
Improve error on timeline view to indicate missing configuration (#197)
Browse files Browse the repository at this point in the history
  • Loading branch information
smarr authored Mar 25, 2024
2 parents e8e027c + 11ab5f5 commit f84de3f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/backend/timeline/timeline.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@
<h1 class="display-4">ReBench: Timeline {%= it.project.name %}</h1>
{% if (it.project.description) { %}
<h2 class="display-5">{%= it.project.description %}</h2>
{% }
if (it.project.basebranch) { %}
<p>Timeline is based on data for the <code>{%= it.project.basebranch %}</code> branch.</p>
{% } %}
</div>
<div class="refresh">
Expand Down Expand Up @@ -80,7 +83,11 @@ <h4><a href="#b-{%= bb.benchName %}-{%= bb.runId %}">{%= bb.benchName %}<span cl
<main role="main">
<div class="jumbotron">
<h1 class="display-4">No Data Available</h1>
<p class="lead">There are no benchmarks available for this project.</p>
{% if (it.project.basebranch) { %}
<p class="lead">There are no benchmarks available for this project.</p>
{% } else { %}
<p class="lead">The branch to show on the timeline has not been configured. Please ask the ReBenchDB administrator to set the branch for the timeline view.</p>
{% } %}
</div>
</div>
</main>
Expand Down
2 changes: 2 additions & 0 deletions tests/data/expected-results/timeline/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
<div class="jumbotron">
<h1 class="display-4">ReBench: Timeline Small Example Project</h1>

<p>Timeline is based on data for the <code>HEAD -&gt; rebenchdb, smarr/rebenchdb</code> branch.</p>

</div>
<div class="refresh">
<div class="flex-nowrap navbar-light">
Expand Down

0 comments on commit f84de3f

Please sign in to comment.