← back to Rentv
fix(rentv-v1): stop review.html linking out to rentvreview.com
fcf02505f00b24495c97628541d0f83da1b0495e · 2026-07-24 08:38:00 -0700 · Steve Abrams
Post-a-Video/Advertise CTAs pointed at rentvreview.com/Account/Register + /ContactUs, and the
video modal 'Open on The REview' used view_url=rentvreview.com/Videos/View. Per the no-rentv-family
link-out rule: CTAs → internal /subscribe + /advertise (both exist), and the video link → YouTube
watch URL (v.yt; external YouTube video links are allowed), hidden when no non-rentvreview target.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Files touched
Diff
commit fcf02505f00b24495c97628541d0f83da1b0495e
Author: Steve Abrams <steve@designerwallcoverings.com>
Date: Fri Jul 24 08:38:00 2026 -0700
fix(rentv-v1): stop review.html linking out to rentvreview.com
Post-a-Video/Advertise CTAs pointed at rentvreview.com/Account/Register + /ContactUs, and the
video modal 'Open on The REview' used view_url=rentvreview.com/Videos/View. Per the no-rentv-family
link-out rule: CTAs → internal /subscribe + /advertise (both exist), and the video link → YouTube
watch URL (v.yt; external YouTube video links are allowed), hidden when no non-rentvreview target.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
public/review.html | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/public/review.html b/public/review.html
index 82089206..914ec52a 100644
--- a/public/review.html
+++ b/public/review.html
@@ -112,8 +112,8 @@
<p>Market analysis, technology, property showcases and industry-event coverage — the free CRE video hub from RENTV.com. Post your own, or advertise to the industry.</p>
</div>
<div class="cta">
- <a class="p" href="https://www.rentvreview.com/Account/Register" target="_blank" rel="noopener">Post a Video</a>
- <a class="g" href="https://www.rentvreview.com/ContactUs" target="_blank" rel="noopener">Advertise</a>
+ <a class="p" href="/subscribe">Post a Video</a>
+ <a class="g" href="/advertise">Advertise</a>
</div>
</div></section>
@@ -191,7 +191,13 @@ function openVideo(id){
document.getElementById('mcat').textContent=v.cat;
document.getElementById('mtitle').textContent=v.title;
document.getElementById('mdesc').textContent=v.desc||'';
- const mlink=document.getElementById('mlink'); if(mlink){ mlink.href=v.view_url||'#'; }
+ // Never link out to rentvreview.com. Videos carry a youtube id (v.yt) — YouTube video links are allowed;
+ // point "Open" at YouTube, hide the link when there's no non-rentvreview target (e.g. RENTV Originals mp4).
+ const mlink=document.getElementById('mlink'); if(mlink){
+ const yt=v.yt?('https://www.youtube.com/watch?v='+v.yt):'';
+ const ext=yt||((v.view_url&&!/rentvreview\.com/i.test(v.view_url))?v.view_url:'');
+ mlink.href=ext||'#'; mlink.textContent=yt?'Watch on YouTube ↗':'Open ↗'; mlink.style.display=ext?'':'none';
+ }
document.getElementById('modal').classList.add('open');
}
function closeVideo(){document.getElementById('modal').classList.remove('open');const box=document.querySelector('.modal .video');if(box)box.innerHTML='';}
← 7a8f7559 fix(rentv-v1): guard non-array feed data in /api/search + /a
·
back to Rentv
·
auto-save: 2026-07-24T08:56:52 (3 files) — data/markets.json d54a5291 →