fix(teched): cross-link on first-class Devtoberfest session concepts (#2312) - #2420
Merged
Merged
Conversation
Release 1.27.0 → main (production)
…2312) The TechEd<->Devtoberfest related-session cross-link matched TechEd concepts against TUTORIAL concepts reached via Activity.TASKSLUG. On PROD the current- edition Devtoberfest sessions reference activities without a usable TASKSLUG, so the join resolved to null and every related array came back empty despite 83 concepts shared between the two session corpora. #2311 made Devtoberfest sessions first-class KG nodes with their own DevtoberfestSessionConceptLinks. Match session<->session on those directly: - loadDevtoberfestSessionsWithConcepts now reads DevtoberfestSessions + DevtoberfestSessionConceptLinks (owned entities; no cross-container planner facade, no edition/activity/tutorial walk). - Forward direction re-keyed by Devtoberfest session ID (buildRelatedTechEdByDtfSession / computeRelatedTechEdByDtfSession); the schedule feed attaches relatedTechEdSessions by session.ID. - Reverse direction (computeRelatedDevtoberfestByTechEd) unchanged in shape, now fed real overlapping concepts. - Dropped the now-unused tutorial-concept loader, DTF facade import, and visible-status filter. Verified against PROD data: reverse 217 TechEd sessions / 481 links, forward 76 Devtoberfest sessions / 209 links (both were 0).
This was referenced Sep 20, 2026
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.
Problem
TechEd ↔ Devtoberfest related-session cross-links render empty on PROD (
relatedTechEdSessions: [],relatedDevtoberfestSessions: []) despite both session corpora being enriched into the KG.Root cause: the crosslink matched TechEd concepts against tutorial concepts reached via
Activity.TASKSLUG. On PROD the current-edition Devtoberfest sessions reference activities with no usableTASKSLUG, so that join resolved to null — even though the two session corpora share 83 concepts.Fix
#2311 made Devtoberfest sessions first-class KG nodes with their own
DevtoberfestSessionConceptLinks. Match session↔session on those directly:loadDevtoberfestSessionsWithConceptsreadsDevtoberfestSessions+DevtoberfestSessionConceptLinks(owned entities — no cross-container facade, no edition/activity/tutorial walk).buildRelatedTechEdByDtfSession/computeRelatedTechEdByDtfSession); schedule feed attaches bysession.ID.computeRelatedDevtoberfestByTechEd) shape unchanged, now fed real overlapping concepts.Verification
teched-devtoberfest-crosslink.test.jsrewritten to the session API +build-teched-endpoint.test.js).Flag-gated (
TECHED_DEVTOBERFEST_CROSSLINK_ENABLED) + fail-open unchanged.