Use the same view code for finished budgets

The differences between the custom code and the code in CONSUL didn't
make much sense anymore: the path can be passed a budget instead of an
ID (so it uses a slug) and there's no reason to hide the link to the
executions for the 2018 budget anymore. Furthermore, removing the
`status: 1` parameter makes the link consistent with the rest of the
application, since it was the only link to budget executions using that
parameter.

app/views/budgets/_finished.html.erb

@@ -16,10 +16,10 @@
               <div class="small-12 medium-6 column table" data-equalizer-watch>
                 <div id="budget_<%= budget.id %>_results" class="table-cell align-middle">
                   <%= link_to t("budgets.index.see_results"),
-                              budget_results_path(budget.id),
+                              budget_results_path(budget),
                               class: "button" %>
                   <%= link_to t("budgets.index.milestones"),
-                              budget_executions_path(budget.id),
+                              budget_executions_path(budget),
                               class: "button" %>
                 </div>
               </div>