Rotating subway-ad campaign on commute station platforms - #413
Open
spencerc99 wants to merge 5 commits into
Open
Rotating subway-ad campaign on commute station platforms#413spencerc99 wants to merge 5 commits into
spencerc99 wants to merge 5 commits into
Conversation
Five posters rotate deterministically per stop domain (FNV-1a over the eligible set): playhtml, the SFPC class, the benches essay, alive internet theory, and the transit pass. The four content ads show for everyone — installed riders included; only the transit-pass ad stays gated on the extension being absent. Tiny platform posters show each ad's real header artwork; clicking opens a readable per-ad overlay in the destination's own palette.
Owner
Author
Preview evidenceTwo-sided station posters at every stop: bigger tiles with real artwork plus a short title; distinct ads per side. Open preview · Review all evidence · Commit Station with posters on both sidesLeft drew the class poster, right the benches essay — distinct ads, deterministic per stop, 108x62 tiles with artwork + title. Open captured page · 1280×900 Right-side ad overlayClicking either side opens that ad's readable card. Open captured page · 1280×900 Published by PR Evidence. Media is temporary and is removed after retention expires. |
Deploying we-were-online-website with
|
| Latest commit: |
57def0a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://6451aaeb.we-were-online-website.pages.dev |
| Branch Preview URL: | https://cd-commute-subway-ads.we-were-online-website.pages.dev |
commit: |
Owner
Author
Code review
|
Each stop hangs a poster on both sides of the station sign; the right slot deterministically draws a different ad than the left. Tiles grow to 108x62 and show the ad's artwork plus a short serif title.
Owner
Author
Code reviewNo issues found. Verified the changes addressing the previous review findings. |
Deploying playhtml with
|
| Latest commit: |
4bfa50b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8515d6fe.playhtml.pages.dev |
| Branch Preview URL: | https://cd-commute-subway-ads.playhtml.pages.dev |
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
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
Follow-up to #410. The single install poster on station platforms becomes a five-ad rotating campaign, each ad wearing the real artwork and palette of what it links to:
Mechanics
Verification
Notes