Skip to content

docs(ui-kit/react): remove loggedInUser prop from v7 MessageList migration reference#410

Merged
ketanyekale merged 2 commits into
mainfrom
docs/v7-docs-update
Jun 26, 2026
Merged

docs(ui-kit/react): remove loggedInUser prop from v7 MessageList migration reference#410
ketanyekale merged 2 commits into
mainfrom
docs/v7-docs-update

Conversation

@shagundeogharkar

Copy link
Copy Markdown
Contributor

Description

  • Remove the loggedInUser prop from the CometChatMessageList "New Props" table in the v7 migration property-changes reference. In v7 this prop is no longer surfaced as a public CometChatMessageList prop, so documenting it under added props was inaccurate.

Related Issue(s)

Type of Change

  • Documentation correction/update
  • New documentation
  • Improvement to existing documentation
  • Typo fix
  • Other (please specify)

Checklist

  • I have read the CONTRIBUTING document
  • My branch name follows the naming convention
  • My changes follow the documentation style guide
  • I have checked for spelling and grammar errors
  • All links in my changes are valid and working
  • My changes are accurately described in this pull request

Additional Information

Verified that no v7 code snippet passes loggedInUser as a prop to any component. Remaining references to loggedInUser in the v7 docs are unrelated and intentionally left in place: the getLoggedInUser() method, the useLoggedInUser() hook, the bubbleView render-prop's second argument, and the PluginContext.loggedInUser field.

Screenshots (if applicable)

@raj-dubey1 raj-dubey1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 pages entries 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 loggedInUser in the file is the bubbleView render-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 loggedInUser as a CometChatMessageList prop.

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

@ketanyekale ketanyekale merged commit e5d0b30 into main Jun 26, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants