Issue-150: Disable Comments disables block notes #151
Merged
Conversation
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
nikkifurls
force-pushed
the
fix/issue-150/disable-comments-block-notes-error
branch
from
c7fb4c0 to
8bec518
Compare
Copilot finished reviewing on behalf of
nikkifurls
November 19, 2025 14:56
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an issue where the Disable Comments feature prevented WordPress 6.9 block notes from being saved or retrieved. The solution enables the REST API comments endpoint for authenticated users while continuing to block it for unauthenticated users to prevent comment spam.
- Modified the authentication check logic to conditionally disable REST API endpoints and comment queries only for unauthenticated users
- Updated documentation to reflect that logged-in users can access comments functionality for block notes
- Bumped plugin version to 3.9.1
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| wp-alleyvate.php | Version bump from 3.9.0 to 3.9.1 |
| src/alley/wp/alleyvate/features/class-disable-comments.php | Added authentication check to conditionally apply filters for REST endpoints and comment queries |
| README.md | Added documentation clarifying that REST API and comment count remain enabled for logged-in users |
| CHANGELOG.md | Added release notes for version 3.9.1 with changes and fixes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
nikkifurls
force-pushed
the
fix/issue-150/disable-comments-block-notes-error
branch
7 times, most recently
from
53a5648 to
3329c91
Compare
nikkifurls
force-pushed
the
fix/issue-150/disable-comments-block-notes-error
branch
from
3329c91 to
8e9fa38
Compare
alley-ci
approved these changes
Nov 19, 2025
srtfisher
approved these changes
Nov 19, 2025
nikkifurls
deleted the
fix/issue-150/disable-comments-block-notes-error
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Fixes #150. Disable Comments disables block notes: The block notes feature in WordPress 6.9 stores notes as comments. Accordingly, the Disable Comments feature in Alleyvate prevents notes from being saved or retrieved. The UI for notes is still visible, making it appear as though a user can write a note. When the user saves the note, an error message appears instead.
Notes for reviewers
We discussed this in #developers and came to the consensus on a solution:
We will enable the REST API comments endpoint, but only provide access to authenticated users, since Notes are always posted in the admin. This fixes the Notes issue, but prevents comment botspam.
Other Information
README.mdfile for any new/updated features.CHANGELOG.mdfile for any new/updated features.Changelog entries
Added
Changed
Deprecated
Removed
Fixed
Security