-
-
Notifications
You must be signed in to change notification settings - Fork 983
TopicsMobileHeader crashes when topic is null聽#15366
Copy link
Copy link
Labels
APP: LearnRe: Learn App (content, quizzes, lessons, etc.)Re: Learn App (content, quizzes, lessons, etc.)DEV: frontendbugBehavior is wrong or brokenBehavior is wrong or brokencommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhelp wantedOpen source contributors welcomeOpen source contributors welcome
Description
Activity
Metadata
Metadata
Assignees
Labels
APP: LearnRe: Learn App (content, quizzes, lessons, etc.)Re: Learn App (content, quizzes, lessons, etc.)DEV: frontendbugBehavior is wrong or brokenBehavior is wrong or brokencommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhelp wantedOpen source contributors welcomeOpen source contributors welcome
馃檪 Looking for an issue? Welcome! This issue is open for contribution. If this is the first time you鈥檙e requesting an issue, please:
Overview
On small screens,
TopicsPagerendersTopicsMobileHeaderbeforetopichas loaded, and the header's render throws aTypeErrorontopic.title.Complexity: Low
Target branch: develop
Context
Production, 90 days to 2026-09-24:
TopicsMobileHeader.vueL13-L22 readstopic.titleandtopic.thumbnailwith no null check.TopicsPageinitialisestopictonull(L364).TopicsHeaderguards each field (L37-L39), butTopicsMobileHeadergetstopicunguarded (L60-L63).The Change
TopicsMobileHeadershould handle anulltopicthe wayTopicsHeaderdoes, until the topic loads.Acceptance Criteria
TopicsMobileHeaderrenders without error whentopicisnull.topicloads, the header shows its title and thumbnail as it does today.TopicsMobileHeaderwithtopic: null.AI usage
I picked this from a Sentry triage with Claude Code. Claude Code queried Sentry, traced the code and drafted the sections.