Skip to content

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

Merged
alexandergull merged 3 commits into
devfrom
spbc_changelog_in_notice.ag
Sep 30, 2026
Merged

alexandergull merged 3 commits into
devfrom
spbc_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.

Copilot review overview

🟡 Changes recommended

Version mismatches can display unrelated release notes, and list styling may be stripped during sanitization.

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

Open (4)
What changed in this PR

Adds changelog details to pending update notices on the WordPress Plugins page.

Changes:

  • Fetches and caches changelogs from WordPress.org or GitHub.
  • Sanitizes and injects changelog markup into update rows.
  • Registers the notice within the admin plugin lifecycle.
File Description
AbstractUpdateChangelogNotice.php Implements fetching, parsing, caching, and injection.
UpdateChangelogNotice.php Configures plugin-specific rendering and escaping.
Escape.php Adds the changelog HTML allowlist.
spbc-admin.php Registers the update notice.

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

Comment thread lib/CleantalkSP/Common/AbstractUpdateChangelogNotice.php Outdated
Comment thread lib/CleantalkSP/Common/AbstractUpdateChangelogNotice.php
Comment thread lib/CleantalkSP/SpbctWP/UpdateChangelogNotice.php
Comment thread lib/CleantalkSP/Common/AbstractUpdateChangelogNotice.php
@svedge
svedge self-requested a review September 30, 2026 05:25
@alexandergull
alexandergull merged commit b2435b5 into dev Sep 30, 2026
7 checks passed
@alexandergull
alexandergull deleted the spbc_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