feat: Featured Snippets, search and filtering for Community page#383
Open
code-snippets-bot wants to merge 29 commits into
Open
feat: Featured Snippets, search and filtering for Community page#383code-snippets-bot wants to merge 29 commits into
code-snippets-bot wants to merge 29 commits into
Conversation
Add get_featured_snippets() to Cloud_API that fetches from the cloud /api/v1/featured endpoint with transient caching derived from the cached_until response field. Add /cloud/featured REST route to expose featured snippets to the frontend.
Load featured snippets on the Community Cloud page when no search query is active. The frontend context fetches from the /cloud/featured REST endpoint on mount and displays results under a "Featured Snippets" heading. When the user performs a search, normal search behaviour takes over. Add PHPUnit tests covering transient caching, cache-hit bypass, and graceful fallback on HTTP errors.
Add 3 additional PHPUnit tests: TTL respects cached_until, missing data key returns empty result, and transient stores Cloud_Snippets instance. Add Playwright E2E spec for community cloud page covering page load, featured heading display, and search override behavior.
louiswol94
approved these changes
May 5, 2026
Contributor
There was a problem hiding this comment.
This PR is clean!
The only suggestion I can make is to mock the responses via Playwright page.route() + route.fulfill() with fixed JSON and X-WP-Total / X-WP-TotalPages headers. That avoids depending on a live cloud/API, removes flakiness from timeouts and env differences, and lets us assert the featured heading, row content, and “search hides featured” etc deterministically. Happy for this to land as a follow-up.
Contributor
|
PlayWright tests are failing |
imantsk
changed the title
imantsk
changed the title
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
3 participants
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
Community Snippets page now shows featured snippets on initial load and supports server-side filtering with category, type, and status dropdowns. Filter options come from the full query result via independent facets.
Featured Snippets
Server-Side Filters
New Endpoints Proxied
Cloud Snippet Model
UI
Test plan