Skip to content

feat: Show required and optional badges in git step #2209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
charlesvien wants to merge 1 commit into
base: 05-18-remove_signals_step_to_improve_conversion
Choose a base branch
from
+19 −9
Open

feat: Show required and optional badges in git step #2209

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 19 additions & 9 deletions apps/code/src/renderer/features/onboarding/components/GitIntegrationStep.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import {
} from "@phosphor-icons/react";
import {
AlertDialog,
Badge,
Box,
Button,
DropdownMenu,
Expand Down Expand Up @@ -206,7 +207,7 @@ export function GitIntegrationStep({
Give your agents access to code
</Text>
<Text className="text-(--gray-11) text-sm">
Point to a local codebase and optionally connect GitHub.
Select a folder to continue. Connecting GitHub is optional.
</Text>
</Flex>
</motion.div>
Expand All @@ -227,11 +228,18 @@ export function GitIntegrationStep({
>
<Flex direction="column" gap="4">
<Flex direction="column" gap="1">
<Flex align="center" gap="2">
<FolderOpen size={18} className="text-(--gray-12)" />
<Text className="font-bold text-(--gray-12) text-base">
Choose your codebase
</Text>
<Flex align="center" justify="between" gap="2">
<Flex align="center" gap="2">
<FolderOpen size={18} className="text-(--gray-12)" />
<Text className="font-bold text-(--gray-12) text-base">
Choose your codebase
</Text>
</Flex>
{selectedDirectory ? (
<Badge color="green">Selected</Badge>
) : (
<Badge color="red">Required</Badge>
)}
</Flex>
<Text className="text-(--gray-11) text-sm">
Select the local folder for your project so we can
Expand Down Expand Up @@ -334,7 +342,7 @@ export function GitIntegrationStep({
className="rounded-[12px] border border-(--gray-a3) bg-(--color-panel-solid)"
>
<Flex direction="column" gap="3">
<Flex align="center" justify="between">
<Flex align="center" justify="between" gap="2">
<Flex align="center" gap="2">
<GitBranch size={18} className="text-(--gray-12)" />
<Text className="font-bold text-(--gray-12) text-base">
Expand Down Expand Up @@ -362,7 +370,9 @@ export function GitIntegrationStep({
</Text>
</Flex>
)
) : null}
) : (
<Badge color="gray">Optional</Badge>
)}
</Flex>
{hasGitIntegration ? (
<Flex direction="column" gap="3">
Expand Down Expand Up @@ -522,7 +532,7 @@ export function GitIntegrationStep({
<Text className="font-bold">
{selectedProject.name}
</Text>
. Sign in with one click to link your account — no
. Sign in with one click to link your account, no
admin approval needed.
</Text>
<Button
Expand Down
Loading