Tomasz Walter
Tomasz Walter
> I wouldn't make the metrics hidden by default as it's all the purpose this report. I think this change is more based on a customer preference. Yes, I agree....
@jbaudoux have your concerns been addressed now?
Hello @jbaudoux can you take a look again and (hopefully) approve?
FTR: I noticed that previous day's moves were not being included in the calculations (although they were supposed to be). The last commit I posted a week ago fixes that...
> @twalter-c2c Can you port the 2 fixes to 16.0 ? @jbaudoux Of course, I will do that next week. Thank you for merging the PR.
@bizzappdev the #2301 has been merged, could you please rebase?
Thank you @bizzappdev @ivs-cetmix @grindtildeath could you please take a look at whether this PR looks good to you?
Functionally looks good.