Change wording to Login using Devant#1528
Conversation
📝 WalkthroughWalkthroughThe login button label in the AIPanel LoginPanel component was updated from "Login to BI Copilot" to "Login using Devant". The underlying functionality and event handler remain unchanged. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
workspaces/ballerina/ballerina-visualizer/src/views/AIPanel/LoginPanel/index.tsx (1)
113-114:⚠️ Potential issue | 🟡 MinorTwo other user-facing "BI Copilot" strings remain unchanged — intentional?
Lines 113–114 (
LegalNotice) and Line 169 (Title) still refer to "BI Copilot":
- Line 113: "BI Copilot uses AI to assist with integration…"
- Line 169: "Welcome to BI Copilot"
If this PR is scoped exclusively to the button label, these are fine. If the intent is a broader rebranding toward "Devant", these strings would also need updating for consistency.
Also applies to: 169-169
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@workspaces/ballerina/ballerina-visualizer/src/views/AIPanel/LoginPanel/index.tsx` around lines 113 - 114, Two remaining user-facing strings in the LoginPanel component still say "BI Copilot" (the LegalNotice text and the Title header); update these string literals in the LoginPanel (index.tsx) to the new brand name (e.g., "Devant") or reference a shared BRAND_NAME constant so all labels stay consistent (update the JSX where LegalNotice and Title are rendered, or extract/replace the hard-coded strings with the shared constant).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In
`@workspaces/ballerina/ballerina-visualizer/src/views/AIPanel/LoginPanel/index.tsx`:
- Around line 113-114: Two remaining user-facing strings in the LoginPanel
component still say "BI Copilot" (the LegalNotice text and the Title header);
update these string literals in the LoginPanel (index.tsx) to the new brand name
(e.g., "Devant") or reference a shared BRAND_NAME constant so all labels stay
consistent (update the JSX where LegalNotice and Title are rendered, or
extract/replace the hard-coded strings with the shared constant).
Purpose
$Subject
Goals
Approach
UI Component Development
npm run storybookfrom the root directory to view current components.Manage Icons
User stories
Release note
Documentation
Training
Certification
Marketing
Automation tests
Security checks
Samples
Related PRs
Migrations (if applicable)
Test environment
Learning
Summary by CodeRabbit