Skip to content

Conversation

@ikreymer
Copy link
Member

To help simplify new UI to avoid separate query

…wlconfig list api (already queried on backend)
@ikreymer ikreymer requested a review from tw4l April 17, 2023 23:43
Copy link
Member

@tw4l tw4l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just to confirm, we don't also need this info in the list endpoint? If we move this into _annotate_with_crawl_stats it'll add it to both endpoints.

Nevermind, was looking at just the last commit!

@ikreymer ikreymer merged commit 821d29b into main Apr 18, 2023
@tw4l tw4l deleted the add-curr-crawl-stats branch April 18, 2023 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants