Move BeyondCode & Laradevs to sponsors in docs sidebar rotation#433
Merged
Conversation
Move the two entries from the partners rotation to the sponsors rotation, and render both docs sidebar lists client-side (Alpine render-all + random pick) via an injectable pool prop, mirroring the ad-rotation component. Tests now cover the rotation mechanism with injected fixtures instead of asserting specific sponsor/partner names. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
simonhamp
marked this pull request as ready for review
July 17, 2026 16:58
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
1 participant
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.
What
Moves BeyondCode and Laradevs from the partners rotation to the sponsors rotation in the docs/blog sidebar.
resources/views/components/sponsors/lists/docs/featured-sponsors.blade.php(partners) — drops the two entries; now rotates Nexcalia, Web Mavens, Synergi Tech.resources/views/components/sponsors/lists/docs/sponsors.blade.php(sponsors) — adds BeyondCode and Laradevs alongside Artisan Build.This completes the sidebar equivalent of #430, which handled the home page.
How
Both sidebar lists previously picked a single entry server-side with
array_rand, which renders only one item per request and can't be tested deterministically (and freezes on the picked entry under full-page caching). They now:x-data+x-show/x-cloak), mirroring the siblingad-rotationcomponent in the same sidebar; and@propsprop (defaulting to the current content), so the data can be fed in — forward-compatible with moving this content to a database later.Tests
Rewrote the placement tests to cover the rotation mechanism with injected fixture pools rather than asserting specific sponsor/partner names (content that's expected to change / move to a DB):
/partnerspage renders its partners section.php artisan test tests/Feature/SponsorsAndPartnersPlacementTest.phpand the relatedAdRotationTestpass; Pint is clean.Note
The partner/sponsor listings are currently duplicated across several views (partners in 3 files, sponsors in 2, in two different data shapes) rather than centralised. Not addressed here; a follow-up could consolidate them into a single source.
🤖 Generated with Claude Code