Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add 1px transparent image in RSS item descriptions for telemetry content views #44

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

guqing
Copy link
Member

@guqing guqing commented Dec 6, 2024

What this PR does?

传统网站即使使用 Umami 等访问统计工具,也难以覆盖 RSS 订阅用户的阅读行为,导致访问量数据不完整,影响对内容表现的准确评估。为解决这一短板,我们新增了在 RSS 订阅中统计访问量的功能支持。

通过在每个 RSS 条目的内容中插入 1 像素透明图片,系统可匿名统计订阅内容的实际阅读量。这种轻量化设计不会影响用户体验,帮助内容创作者更全面的了解内容阅读量情况。

Umami 适配参考 halo-sigs/plugin-umami#30

为 RSS 订阅内容统计访问量提供扩展支持,本插件并不提供任何存储和分析访问量的功能但允许其他插件扩展并获取访问量数据上报给诸如 Umami 之类的应用

@f2c-ci-robot f2c-ci-robot bot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Dec 6, 2024
@f2c-ci-robot f2c-ci-robot bot requested review from JohnNiang and wan92hen December 6, 2024 09:12
@guqing
Copy link
Member Author

guqing commented Dec 8, 2024

/hold

@f2c-ci-robot f2c-ci-robot bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 8, 2024
@guqing
Copy link
Member Author

guqing commented Dec 10, 2024

/unhold

@f2c-ci-robot f2c-ci-robot bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 10, 2024
Copy link
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

/lgtm

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2024
Copy link

f2c-ci-robot bot commented Dec 10, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ruibaby

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@f2c-ci-robot f2c-ci-robot bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 10, 2024
@f2c-ci-robot f2c-ci-robot bot merged commit f6b461d into main Dec 10, 2024
2 checks passed
@guqing guqing deleted the feature/telemetry branch December 10, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants