mirror of
https://github.com/harvard-edge/cs249r_book.git
synced 2026-07-21 00:23:30 -05:00
[GH-ISSUE #1844] staffml: Subscribe link in EcosystemBar points at a non-existent #subscribe anchor #35281
Closed
opened 2026-07-15 22:41:24 -05:00 by GiteaMirror
·
2 comments
No Branch/Tag Specified
dev
audit/mlperf-edu-readiness
feat/mlperf-edu-precondition
gh-pages
vol1/all-final
main
vol1/appendices-final
vol1/ch16-final
vol1/ch15-final
vol1/ch14-final
vol1/ch13-final
vol1/ch11-final
vol1/ch12-final
vol1/ch10-final
vol1/ch9-final
vol1/ch8-final
vol1/ch7-final
vol1/ch6-final
vol1/ch5-final
vol1/ch4-final
vol1/ch3-final
vol1/ch2-final
vol1/frontmater-final
kai/fixing-profile-setting-and-map
chore/staffml-ci-path
fix/callout-flow
vol1/ch10-pass4
vol1/ch9-pass4
vol1/ch8-pass4
vol1/ch7-pass4
vol1/ch6-pass4
vol1/ch5-pass4
vol1/apC-pass3
vol1/ch4-pass4
vol1/ch3-pass4
vol1/ch2-pass4
vol1/ch1-pass4
vol1/frontmatter
vol1/apE-pass3
vol1/apD-pass3
fmt-fix
vol1/ch14-pass3
kai/clarify-community-map-totals
vol1/ch13-pass3
vol1/ch12-pass3
vol1/ch11-pass3
vol1/ch10-pass3
vol1/ch7-pass3
vol1/ch9-pass3
vol1/ch8-pass3
vol1/ch6-pass3
vol1/ch5-pass3
vol1/ch4-pass3
vol1/ch3-pass3
vol1/ch2-pass3
vol1/ch1-pass3
vol1/ch6-pass2
vol1/ch5-pass2
vol1/ch4-pass2
vol1/ch3-pass2
vol1/ch2-pass2
fix/badge-fixes
chore/precommit-cleanup
cleanup/book-validate-paths
fix/staffml-trigger-on-workflow-edits
fix/staffml-reusable-concurrency
feat/container-preflight-urls
instructors-v0.1.1
vol1-pr1957-merged
vol2-v0.2.1
vol1-v0.7.1
vol1-v0.7.0+vol2-v0.2.0
slides-v0.1.0
vol2-v0.2.0
vol1-v0.7.0
tinytorch-v0.1.13
mlsysim-v0.2.0
vol2-v0.1.2
vol1-v0.6.2
tinytorch-v0.1.12
vol2-v0.1.1
vol1-v0.6.1
mlsysim-v0.1.3
vol2-v0.1.0
vol1-v0.6.0
mlsysim-v0.1.2
staffml-v0.1.1
tinytorch-v0.1.11
labs-v0.1.0
kits-v0.1.0
slides-latest
instructors-v0.1.0
staffml-v0.1.0
mlsysim-v0.1.1
mlsysim-v0.1.0
tinytorch-v0.1.10
build-verified-windows-v1
tinytorch-v0.1.9
tinytorch-v0.1.8
mit-submission-v1
tinytorch-v0.1.7
tinytorch-v0.1.6
tinytorch-v0.1.5
tinytorch-slides-v0.1.0
tinytorch-v0.1.3
tinytorch-v0.1.4
tinytorch-v0.1.2
tinytorch-v0.1.1
tinytorch-v0.1.0
book-v0.5.1
tinytorch-audio-v0.1.1
tinytorch-audio-assets-v0.1
book-v0.5.0
book-v0.4.2
book-v0.4.1
book-v0.4.0
book-v0.3.0
book-v0.2.0
book-v0.1.0
Labels
Clear labels
area: book
area: collabs
area: kits
area: labs
area: socratiq
area: staffml
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
javascript
link-health
link-rot
priority-high
pull-request
staffml
type: bug
type: citation
type: code
type: errata
type: improvement
type: new
type: question
vault-sli
Mirrored from GitHub Pull Request
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/cs249r_book#35281
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Originally created by @farhan523 on GitHub (Jun 6, 2026).
Original GitHub issue: https://github.com/harvard-edge/cs249r_book/issues/1844
Bug
The Subscribe link in the StaffML
EcosystemBarpoints at#subscribeon both desktop and mobile, but no element withid=\"subscribe\"exists anywhere ininterviews/staffml/. Clicking it appends#subscribeto the URL and does nothing — silent dead-end.interviews/staffml/src/components/EcosystemBar.tsx#L492interviews/staffml/src/components/EcosystemBar.tsx#L620grep -r 'id=\"subscribe\"' interviews/staffml/src→ no matchesWhy this happened (best guess)
Commit
8b1c235f26 fix(staffml): complete navbar parity with navbar-common.ymlported the navbar structure fromshared/config/navbar-common.yml. That YAML has BOTH:…and a separate Newsletter entry:
The
#subscribeanchor on the main Quarto site is presumably wired to a JS popover (subscribe modal) that opens when clicked. StaffML is a separate Next.js app and inherits the link target but not the popover wiring, so the link dangles.Suggested fix
Point the StaffML Subscribe link at
https://mlsysbook.ai/newsletter/— that's already the canonical newsletter URL in bothshared/config/footer-site.ymland theConnect → community → newsletterentry inshared/config/navbar-common.yml. One-line change in two places inEcosystemBar.tsx.Alternative paths if the newsletter URL isn't right:
#subscribesection to the page that opens it.What I'd like from you
A one-line answer: what's the right target? I'll send a PR with whatever you confirm. Thanks! @profvjreddi
@profvjreddi commented on GitHub (Jul 2, 2026):
Thanks @farhan523, fixed. The EcosystemBar Subscribe link (desktop and mobile) was pointing at a non-existent
#subscribeanchor; it now points to https://mlsysbook.ai/newsletter/, matching the shared navbar config, which also documents why an internal#subscribeanchor is avoided (Safari content blockers hide anchors whose href contains "subscribe"). It will go live with the next StaffML deploy.@all-contributors please add @farhan523 for 🪲 Bug in StaffML
@github-actions[bot] commented on GitHub (Jul 2, 2026):
I've added @farhan523 as a contributor to staffml! 🎉
Recognized for: bug, code
Project(s): staffml (explicitly mentioned in comment)
Based on: @all-contributors please add @farhan523 for 🪲 Bug in StaffML
The contributor list has been updated in:
interviews/.all-contributorsrc,interviews/README.mdREADME.mdWe love recognizing our contributors! ❤️