Traffic: show 200 and 404 pages

readthedocs / ext-theme

Read the Docs drop in replacement site templates

2 stars 2 forks source link

Traffic: show 200 and 404 pages #364

Closed humitos closed 3 months ago

humitos commented 3 months ago

This PR makes the minimal changes to show the data when it's available. There may be some tweaks and styles we want to improve, but I focus on making the data to be shown first. We can come back later and improve it in a second iteration.

Screenshot 2024-06-05 at 18-40-37 Traffic Analytics - Read the Docs

Closes #349

agjohnson commented 3 months ago

Yeah, to follow the patterns on all the other listing UIs, this should be a filter, not two separate tables. We should return to this to fix this, as I don't want to break consistency in our UI/template code.

https://github.com/readthedocs/ext-theme/issues/366

humitos commented 3 months ago

this should be a filter, not two separate tables

Sounds like a good idea to me 👍🏼