Skip to content

New. Plugins page. Show changelog in the update notice. - #901

Merged
alexandergull merged 2 commits into
devfrom
apbct_changelog_in_notice.ag
Sep 30, 2026
Merged

alexandergull merged 2 commits into
devfrom
apbct_changelog_in_notice.ag

Conversation

@alexandergull

Copy link
Copy Markdown
Member

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Warning

  • Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.

Copilot review overview

🟡 Changes recommended

It can break PHP 5.6 admin requests and may display changelogs for the wrong plugin version.

Review effort: Balanced
Findings: 1 Medium severity · 1 Low severity

Open (2)
What changed in this PR

Adds update changelogs to the native WordPress plugin update notice.

Changes:

  • Fetches and caches changelogs from WordPress.org or GitHub.
  • Sanitizes and renders changelog markup.
  • Registers the feature in the admin area.
File Description
lib/​Cleantalk/​Common/​AbstractUpdateChangelogNotice.php Implements changelog retrieval, parsing, caching, and injection.
lib/​Cleantalk/​ApbctWP/​UpdateChangelogNotice.php Configures the feature for CleanTalk.
lib/​Cleantalk/​ApbctWP/​Escape.php Adds changelog-specific KSES markup.
inc/​cleantalk-admin.php Registers the changelog notice.
tests/​ApbctWP/​EscapeTest.php Test changes excluded from review.
tests/​Common/​AbstractUpdateChangelogNoticeTest.php Test changes excluded from review.
Files excluded by content exclusion policy (2)
  • tests/ApbctWP/EscapeTest.php
  • tests/Common/AbstractUpdateChangelogNoticeTest.php

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lib/Cleantalk/ApbctWP/Escape.php
Comment thread lib/Cleantalk/Common/AbstractUpdateChangelogNotice.php Outdated
@codecov

codecov Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 20.97561% with 162 lines in your changes missing coverage. Please review.
✅ Project coverage is 31.96%. Comparing base (b6945e5) to head (01c37ad).

Files with missing lines Patch % Lines
...Cleantalk/Common/AbstractUpdateChangelogNotice.php 21.69% 148 Missing ⚠️
lib/Cleantalk/ApbctWP/UpdateChangelogNotice.php 0.00% 14 Missing ⚠️

❌ Your patch check has failed because the patch coverage (20.97%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #901      +/-   ##
============================================
- Coverage     32.03%   31.96%   -0.08%     
- Complexity     6781     6873      +92     
============================================
  Files           292      294       +2     
  Lines         26775    26980     +205     
============================================
+ Hits           8578     8623      +45     
- Misses        18197    18357     +160     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alexandergull
alexandergull force-pushed the apbct_changelog_in_notice.ag branch from 0b4155e to 772b91a Compare September 29, 2026 13:12
@alexandergull
alexandergull merged commit 80f4286 into dev Sep 30, 2026
8 of 9 checks passed
@alexandergull
alexandergull deleted the apbct_changelog_in_notice.ag branch September 30, 2026 12:49
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.

3 participants