Closed JoHeyJo closed 11 months ago
@mikeubell how's this? I only made changes to process.rb
All the other file changes, I believe, are due to me running make process
.
ahh. yes. ok!
@mikeubell how does this look? I moved the logic over to election_totals.rb
. This is a much cleaner approach and data isn't needlessly passed around waiting to get filtered out. Thanks for pointing that out!
Great! Just want to double-check since this is my first PR with this group. Now that it's approved am I good to merge?
You are good to go.On Sep 26, 2023, at 5:22 PM, JoHeyJo @.***> wrote: Great! Just want to double-check since this is my first PR with this group. Now that it's approved am I good to merge?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>
Races with zero amount(contributions) are filtered out of "most_expensive_races". Only races with amount > 0 remain in the object to be passed along to the frontend.