Skip to content

docs: add video stats to content/slack endpoint#82

Merged
sweetmantech merged 1 commit intomainfrom
docs/content-slack-video-stats
Mar 26, 2026
Merged

docs: add video stats to content/slack endpoint#82
sweetmantech merged 1 commit intomainfrom
docs/content-slack-video-stats

Conversation

@sweetmantech
Copy link
Copy Markdown
Collaborator

@sweetmantech sweetmantech commented Mar 26, 2026

Summary

  • Add total_videos and tags_with_videos fields to the GET /api/admins/content/slack response schema
  • Enables measuring tag-to-video conversion rate, mirroring the coding endpoint's PR stats pattern

Test plan

  • Verify OpenAPI spec renders correctly in Mintlify
  • Confirm API implementation returns the new fields when deployed

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • New Features

    • Admin Slack content reports now include aggregate video statistics: total videos generated and count of tags that produced videos.
  • Documentation

    • Updated API documentation to reflect new report fields.

Add aggregate video statistics to measure tag-to-video conversion,
mirroring the coding/slack endpoint's PR stats pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 481ff76c-a7e8-467a-b458-e49064dded1e

📥 Commits

Reviewing files that changed from the base of the PR and between 41ba213 and 4b844ba.

📒 Files selected for processing (1)
  • api-reference/openapi.json

📝 Walkthrough

Walkthrough

The pull request updates the OpenAPI specification for the /api/admins/content/slack endpoint, adding two new required fields—total_videos and tags_with_videos—to the response schema to include aggregate video conversion statistics alongside existing per-tag data.

Changes

Cohort / File(s) Summary
OpenAPI Schema Update
api-reference/openapi.json
Updated the GET /api/admins/content/slack response schema to require two new integer fields: total_videos (total videos across all tags) and tags_with_videos (count of tags with videos). Updated required properties array and endpoint description.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 Hops through the API with glee,
New fields added, one, two, three!
Total videos, tags that glow,
Our schema's stats now steal the show!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/content-slack-video-stats

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sweetmantech sweetmantech merged commit 125afef into main Mar 26, 2026
1 of 2 checks passed
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.

1 participant