Skip to content

Add Messages Overview tab to Data Analytics docs - #684

Closed
rvignesh89 wants to merge 3 commits into
mainfrom
claude/glific-docs-update-skill-cb0216
Closed

rvignesh89 wants to merge 3 commits into
mainfrom
claude/glific-docs-update-skill-cb0216

Conversation

@rvignesh89

Copy link
Copy Markdown
Contributor

Summary

The Data Analytics dashboard now has four tabs, but the docs listed only three. This PR documents the missing Messages Overview tab, based on the live production dashboard.

docs/4. Product Features/Data Analytics.md

  • Intro now lists four tabs: Reach & Growth, Engagement Quality, Messages Overview, Message Errors.
  • Replaced the intro screenshot (GitHub CDN image) with a new local one that shows all four tabs.
  • New Messages Overview section covering:
    • the Messages Exchanged card (Inbound / Template Outbound / Service Outbound, and the "Billable from 1st Oct" tag);
    • the notice about Meta charging for service messages from 1st October 2026;
    • the Message Composition stacked bar chart.
  • Header: Last Updated → October 2026, read time 3 → 4 minutes.

New images (static/img/analytics/), taken with the redacting gdocs shot: the staff name is redacted, and the rest shows only aggregate counts.

  • analytics_dashboard_tabs.png
  • analytics_messages_overview.png

Notes

  • This branch also carries the two earlier update-docs skill commits that aren't on main yet (c614abf, fc6d792).
  • gdocs lint passes. node_modules wasn't installed locally, so CI will run the Docusaurus build.

🤖 Generated with Claude Code

https://claude.ai/code/session_01YZEqcwqazipcxQbcraCE5G


Generated by Claude Code

rvignesh89 and others added 3 commits October 10, 2026 11:17
/update-docs writes, refreshes, reviews and restructures docs against the
live Glific app from a cloud env with playwright-cli preinstalled.

- gdocs.mjs (zero-dep): preflight (env, CLI, browser, guard canary,
  redaction self-test, login), guarded session, redacted screenshots,
  blocked-write log, provenance/link lint
- Network guard blocks mutations, lock-taking queries (flowGet,
  simulatorGet) and non-GET REST writes before login
- Bash hooks restrict direct playwright-cli use to the guarded session
  and surface blocked writes so Claude stops and warns

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Move sections, style-writing, style-page and style-images into
update-docs and remove the old documentation skill.

- Header guidance now matches the <h3><table> block with Last Updated
  that most pages use, instead of the legacy ###** badge
- Image guide requires gdocs shot; adds framing and filename-stability rules
- Sections: _category_.json generated-index (not index.md), more routes
- Refresh workflow accepts a glific-frontend PR as input

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Document the new Messages Overview tab (Messages Exchanged card,
service message billing notice, Message Composition chart) and
replace the intro screenshot with one showing all four tabs.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01YZEqcwqazipcxQbcraCE5G
@coderabbitai

coderabbitai Bot commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions

Copy link
Copy Markdown

This branch was previously deployed

1 inactive deployment
pull request — ff469fe0 Deployed Oct 10, 2026 by github-actions[bot]
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.

2 participants