Uh oh!
There was an error while loading. Please reload this page.
feat: Add Buzz workspace dashboard with charts and cards - #313
Merged
Conversation
The Buzz workspace shipped empty ("content": "[]") even though hooks.py
points the apps screen at /app/buzz, so every site installing buzz landed
on a blank page.
Add two standard Number Cards (Total Events, Active Events) and a
standard "Events by Category" Group By chart under the Events module.
Being standard + module-scoped, they are picked up by sync_dashboards()
on both install_app and bench migrate, so any site that installs buzz
gets them without extra hooks or fixtures.
Wire them into the workspace along with three shortcuts: two URL
shortcuts that prefill the category on the new-event form, and a plain
New Event shortcut. Shortcuts cannot carry route options, so prefilling
has to go through the URL form.
Active Events uses dynamic_filters_json for the end_date comparison so
the cutoff is evaluated at render time rather than frozen at authoring
time.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01437BfuRXkj3k8JsQXMr3YJContributor
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
The "Events by Category" donut rendered an unreadable legend. frappe-charts lays legend items out at a fixed x = LEGEND_ITEM_WIDTH * count (150px slots, BaseChart.js renderLegend) but never clips the label text to its slot, and frappe core passes truncateLegends: 0, so longer category names ran straight over their neighbours and the last item spilled past the card. Switch the chart to Bar. Axis charts carry their labels on the x-axis instead of a legend row, so the overlap cannot happen, and the bars stay readable when one category dominates the counts — which a donut does not. number_of_groups was 0, which the widget reads as "unset" and falls back to max_slices: 6 for Pie/Donut, bucketing real categories into "Rest". Bar falls back to 9, but set it explicitly so the value does not depend on the chart type. Also add New Zoom Meeting and New Zoom Webinar quick actions. There is no "Zoom Webinar" category — Webinars and Zoom Meeting are the two Zoom-backed ones, paired in the free_event depends_on and behind create_webinar_on_zoom / create_meeting_on_zoom — so the webinar shortcut targets Webinars. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Contributor
Successfully created backport PR for |
harshtandiya added a commit
that referenced
this pull request
Aug 3, 2026
feat: Add Buzz workspace dashboard with charts and cards (#313) * feat(events): populate Buzz workspace with cards, chart and shortcuts The Buzz workspace shipped empty ("content": "[]") even though hooks.py points the apps screen at /app/buzz, so every site installing buzz landed on a blank page. Add two standard Number Cards (Total Events, Active Events) and a standard "Events by Category" Group By chart under the Events module. Being standard + module-scoped, they are picked up by sync_dashboards() on both install_app and bench migrate, so any site that installs buzz gets them without extra hooks or fixtures. Wire them into the workspace along with three shortcuts: two URL shortcuts that prefill the category on the new-event form, and a plain New Event shortcut. Shortcuts cannot carry route options, so prefilling has to go through the URL form. Active Events uses dynamic_filters_json for the end_date comparison so the cutoff is evaluated at render time rather than frozen at authoring time. Claude-Session: https://claude.ai/code/session_01437BfuRXkj3k8JsQXMr3YJ * fix(events): make the category chart readable and add Zoom shortcuts The "Events by Category" donut rendered an unreadable legend. frappe-charts lays legend items out at a fixed x = LEGEND_ITEM_WIDTH * count (150px slots, BaseChart.js renderLegend) but never clips the label text to its slot, and frappe core passes truncateLegends: 0, so longer category names ran straight over their neighbours and the last item spilled past the card. Switch the chart to Bar. Axis charts carry their labels on the x-axis instead of a legend row, so the overlap cannot happen, and the bars stay readable when one category dominates the counts — which a donut does not. number_of_groups was 0, which the widget reads as "unset" and falls back to max_slices: 6 for Pie/Donut, bucketing real categories into "Rest". Bar falls back to 9, but set it explicitly so the value does not depend on the chart type. Also add New Zoom Meeting and New Zoom Webinar quick actions. There is no "Zoom Webinar" category — Webinars and Zoom Meeting are the two Zoom-backed ones, paired in the free_event depends_on and behind create_webinar_on_zoom / create_meeting_on_zoom — so the webinar shortcut targets Webinars. --------- (cherry picked from commit fbc6315) Co-authored-by: Harsh Tandiya <harsh.tandiya@gmail.com> Co-authored-by: Claude <noreply@anthropic.com>
NagariaHussain
commented
Aug 3, 2026
Contributor
@harshtandiya please add screenshots for all UI related PRs! |
harshtandiya
commented
Aug 3, 2026
CollaboratorAuthor
@NagariaHussain added |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Populate the Buzz workspace with dashboard components including charts, number cards, and quick action shortcuts to provide users with an overview of events and quick access to common actions.
Key Changes
Workspace Configuration (
buzz.json):New Dashboard Chart (
events_by_category.json):New Number Cards:
active_events.json): Displays count of published events with end_date >= today, colored green (#29CD42)total_events.json): Displays total count of all events with percentage stats, colored blue (#449CF0)Implementation Details
Buzz Eventdocument type as their data sourcehttps://claude.ai/code/session_01437BfuRXkj3k8JsQXMr3YJ