Feat/issue engagement table - #6
Merged
Merged
Conversation
Drops the manual column alignment in loadStats and regaps the diskData struct tags. Whitespace only, so `gofmt -l .` is clean and the CI formatting check has nothing to flag on future changes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The Issues tab previously required typing an issue number before it showed anything. It now leads with a table of the last 10 issues carrying total clicks, total opens, open rate, and click rate, with the link breakdown below it. Rows are clickable and highlight the selected issue in both the table and the overview trend chart. /api/issues gains date, deliveries, opens, open_rate, and click_rate, sourced from the analytics endpoint through the existing disk cache the Trends tab already warms. Analytics failures are best effort: they log and leave the fields zeroed, and the dashboard renders a dash rather than a misleading 0% when deliveries is 0. Issue totals now skip excluded domains via sumIncluded. The summary counted buttondown.com clicks while the drilldown filtered them, so the table and the links below it disagreed. Clicks and click rate deliberately do not divide into each other: clicks counts every click event, while click rate uses Buttondown's own per-subscriber figure. A note under the table explains it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.