Skip to content

Brand own space as Nova chats#957

Merged
Dhravya merged 1 commit into
mainfrom
Dhravya/nova-chats-logo
May 17, 2026
Merged

Brand own space as Nova chats#957
Dhravya merged 1 commit into
mainfrom
Dhravya/nova-chats-logo

Conversation

@Dhravya
Copy link
Copy Markdown
Member

@Dhravya Dhravya commented May 17, 2026

Renames the user-owned conversation space from "Your conversations" to "Nova chats".
Uses the existing Nova orb logo for that own-space entry in the space selector trigger, modal list, and delete target selector.

Testing: bunx biome check apps/web/lib/ingest-auto-space.ts apps/web/components/space-selector.tsx apps/web/components/select-spaces-modal.tsx

@claude
Copy link
Copy Markdown

claude Bot commented May 17, 2026

Claude finished @Dhravya's task in 2m 15s —— View job


Reviewed PR #957: Brand own space as Nova chats

  • Fetch and analyze the diff
  • Review changed files for bugs/security issues
  • Check for existing review comments
  • Submit review

Overview: Renames the user's own conversation space to "Nova chats" and displays the NovaOrb icon instead of the default emoji.

Issues found: None — this looks good to ship.

Verified:

  • isOwnConversationSpace logic correctly matches containerTag === userId
  • All imports added properly (NovaOrb, isOwnConversationSpace)
  • useMemo dependency array includes user?.id as required
  • No hardcoded "Your conversations" strings remaining in codebase
  • Biome check passes

Score: 10/10

@Dhravya Dhravya merged commit fd78633 into main May 17, 2026
5 of 7 checks passed
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Overview: Renames the user's own conversation space to "Nova chats" and displays the NovaOrb icon instead of the default emoji.

Issues found: None — this looks good to ship.

Verified:

  • isOwnConversationSpace logic correctly matches containerTag === userId
  • All imports added properly (NovaOrb, isOwnConversationSpace)
  • useMemo dependency array includes user?.id as required
  • No hardcoded "Your conversations" strings remaining in codebase
  • Renamed constant OWN_CHAT_SPACE_NAME is only used internally
  • Biome check passes

Score: 10/10

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