Here are some screenshots

Welp...

15:43:06  Failed tests:
15:43:06  FAILED test: py3-test_pagure_flask_api_issue

Would it be possible to add a 3rd item on the tickets: 'total open' ie, the number of open tickets at the start of the week? this would allow you to see easier if backlog was increasing or declining over time.

@kevin I think this was the original idea of this graph but doesn't quite work as expected, I may have an idea though, let's see if that'll work :)

@kevin how does this look?

(these are the pagure issues, I'm working on the infra ones to see if they work as well)

Wow, that looks great! : D

@kevin how does this look?

Looks great

This looks fantastic!

:thumbsup:

Awesome! The only nitpick I might have is that I am not sure blue and pink are good colors to use here... not sure what is tho...

great work!

It's actually blue and red, but the alpha layer make the red pinkish yeah :)

I'll see if I can make it a little darker to increase the contrast

Now with unit-tests for the change to the API endpoint.

I've tweaked a little bit the color, but I doubt it's ideal yet, though I figure we can always tweak it more later :)

I don't fully understand this apidoc. Whitout detailed returns info for that week but with detailed for the last 53 weeks? Seems like a c&p glitch :)

It always returns the last 53 weeks, the detailed view as just more info than the non-detailed view.
Tbh, if it was me I'd make the default view be the detailed one, but that would be breaking API so no-go :(

Maybe it would make sense to have a new API endpoint for this, so we can simply remove the old one at some point in the future.

this could be _be instead of callling jquery's selector again? _b already has #commiter-list block from line 155

we were using the d3's minimized version, but now we include Chart.js's non minimized one. Should we import the minimized one?

does the heatmap thing depend on d3.v4.min.jis that this pr removes? (did not test it) or it conitnues working after this?

fair :thumbsup:

We have both version for chartjs, we can use the minimized one sure

:thumbsup:

:thumbsup:

lgtm once Jenkins is happy

Jenkins is happy, I'm happy people like the screenshots.

So let's get this in!