docs(ui-kit/react): remove loggedInUser prop from v7 MessageList migration reference#410
Conversation
raj-dubey1
left a comment
There was a problem hiding this comment.
Docs review — approve ✅
TL;DR: A one-line correction that removes a single inaccurate row (loggedInUser) from the CometChatMessageList → New Props table in ui-kit/react/migration-property-changes.mdx. It touches no file paths, navigation, links, or images, so there is zero risk of 404s, build breakage, or link rot. The deletion is accurate and the surrounding content stays consistent.
What passed (actually checked)
- Redirect coverage — safe: content-only modification; no files moved, renamed, or deleted, so no URL dies and no redirect is needed.
- Navigation integrity — safe: no
pagesentries affected; the build cannot break from this change. - In-content link rot — safe: no links added, removed, or repointed.
- Content integrity — correct:
- The removed row was correctly located under
## CometChatMessageList→### New Props. The remaining table is internally consistent. - The only surviving
loggedInUserin the file is thebubbleViewrender-prop's callback argument ((message, loggedInUser) => ReactNode) — that's a callback parameter, not a component prop, so it is correctly left in place. This matches the PR description. - No off-by-one risk: there is no "N new props" count statement near the section, and no prose elsewhere in the file describes
loggedInUseras aCometChatMessageListprop.
- The removed row was correctly located under
Findings
P0 / P1 / P2: none.
Scope note
Verified the one changed file end-to-end. The repo-wide redirect/orphan script wasn't run (a depth-1 branch clone timed out), but it isn't relevant here since this PR adds/removes/renames no files. Other v7 loggedInUser references called out in the PR description (getLoggedInUser(), useLoggedInUser(), PluginContext.loggedInUser) live in other files and are outside this diff.
🤖 Reviewed with Claude Code
Description
loggedInUserprop from theCometChatMessageList"New Props" table in the v7 migration property-changes reference. In v7 this prop is no longer surfaced as a publicCometChatMessageListprop, so documenting it under added props was inaccurate.Related Issue(s)
Type of Change
Checklist
Additional Information
Verified that no v7 code snippet passes
loggedInUseras a prop to any component. Remaining references tologgedInUserin the v7 docs are unrelated and intentionally left in place: thegetLoggedInUser()method, theuseLoggedInUser()hook, thebubbleViewrender-prop's second argument, and thePluginContext.loggedInUserfield.Screenshots (if applicable)