Skip to content

Learn: remove the Udemy call to action; header shows the signed-in member - #562

Merged
wienke merged 2 commits into
masterfrom
docs/remove-udemy-cta
Oct 9, 2026
Merged

wienke merged 2 commits into
masterfrom
docs/remove-udemy-cta

Conversation

@wienke

@wienke wienke commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Two changes requested after the www cutover (2026-10-09):

  • LoRaWAN intro (content/lorawan/_index.md): remove the info box "The LoRaWAN Fundamentals course can also be followed on Udemy. Sign up for the online course today!"
  • Header shows the signed-in member. The global header always showed Log in / Sign up, even for a signed-in member. The auth block is now a data-ttn-auth="dx" hook for www's /site-session.js (served by the thethingsnetwork-v2 platform, like /consent.js). That script asks www's /api/session and renders the same account menu as the rest of www (avatar, username, My profile, Settings, Console, Log out) in the docs' own dx- classes. Signed out, or outside the www front door, nothing changes. docs.css gets the small rules for the right-aligned menu and the Log out button.

🤖 Generated with Claude Code

wienke and others added 2 commits October 9, 2026 09:56
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The global header always showed Log in / Sign up, also for a signed-in member. The auth block is now a data-ttn-auth="dx" hook for www's /site-session.js (thethingsnetwork-v2 platform), which renders the same account menu as www in the docs' dx- classes; signed out it stays as is. docs.css: right-aligned account menu, summary marker hidden, Log out button styled as a menu item.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@wienke
wienke requested a review from pierrephz as a code owner October 9, 2026 08:00
@wienke
wienke merged commit 8a46574 into master Oct 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant